Skip to content
This repository has been archived by the owner on Apr 21, 2024. It is now read-only.

!Pulse now checks the discord status api #53

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

chevyboys
Copy link
Contributor

pulse will now be able to get discord's component operating status.

pulse will now be able to get discord's component operating status.
Comment on lines +150 to +151
fetch("https://srhpyqt94yxb.statuspage.io/api/v2/summary.json").then(res => res.json())
.then((json) => {
Copy link
Contributor

Choose a reason for hiding this comment

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

fetch isn't defined anywhere. For consistency sake, recommend requiring axios at the top and using that.

In addition, using .then() in an async function is generally seen as bad form. Better to use await here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

oh, I was using node fetch. Sorry about that. I don't know axios.

Comment on lines +181 to +182
fetch("https://srhpyqt94yxb.statuspage.io/api/v2/summary.json").then(res => res.json())
.then((json) => {
Copy link
Contributor

Choose a reason for hiding this comment

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

See note @ 150-151.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants