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
Copy link
Copy Markdown
Contributor

@tnfAngel tnfAngel commented Dec 9, 2023

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
Copy link
Copy Markdown
Contributor Author

tnfAngel commented Dec 9, 2023

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
Copy link
Copy Markdown

@LautyDev LautyDev left a comment

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