Skip to content

Fix STW News#747

Draft
tnfAngel wants to merge 3 commits into
fnbrjs:mainfrom
tnfAngel:stw-news
Draft

Fix STW News#747
tnfAngel wants to merge 3 commits into
fnbrjs:mainfrom
tnfAngel:stw-news

Conversation

@tnfAngel

@tnfAngel tnfAngel commented Dec 9, 2023

Copy link
Copy Markdown
Contributor

Fix STW News
I can't test the STW endpoint because I don't have STW and there is no news, so I tried it with sparks-motd
(STW_NEWS_MOTD: 'https://prm-dialogue-public-api-prod.edea.live.use1a.on.epicgames.com/api/v1/fortnite-br/surfaces/sparks-motd/target') and it works fine

I hope everything is correct

@tnfAngel

tnfAngel commented Dec 9, 2023

Copy link
Copy Markdown
Contributor Author

solves #745

@tnfAngel tnfAngel marked this pull request as draft December 9, 2023 21:22
@ThisNils ThisNils linked an issue Dec 9, 2023 that may be closed by this pull request
@tnfAngel tnfAngel marked this pull request as ready for review December 9, 2023 21:34

@LautyDev LautyDev left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! 😺

@LeleDerGrasshalmi

Copy link
Copy Markdown
Contributor

@tnfAngel, you may wanna update to the new news api, the one in the PR has recently been removed.

@tnfAngel tnfAngel marked this pull request as draft November 5, 2024 15:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

STW News not working

3 participants