-
Notifications
You must be signed in to change notification settings - Fork 301
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update wmn-data.json #906
Update wmn-data.json #906
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please address comments and thanks for the submission!
wmn-data.json
Outdated
"name" : "Statuspage", | ||
"uri_check" : "https://{account}.statuspage.io/api/v2/status.json", | ||
"e_code" : 200, | ||
"e_string" : "id", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This e_string
is a bit too plain for us as it could appear on page when a profile is not present. Consider replacing with updated_at
wmn-data.json
Outdated
"e_string" : "id", | ||
"m_string" : "<html><body>You are being <a href=\"https://www.statuspage.io\">redirected</a>.</body></html>", | ||
"m_code" : 302, | ||
"known" : ["8713981tpdlg"], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add 1-2 more known
accounts. I found a bunch by Googling https://www.google.com/search?q=site%3Astatuspage.io&oq=site%3Astatuspage.io
Amened account identification values and added pretty url.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please review the new changes.
wmn-data.json
Outdated
@@ -6602,6 +6602,17 @@ | |||
"m_code" : 404, | |||
"known" : ["carlito", "taylor"], | |||
"cat" : "social" | |||
}, | |||
{ | |||
"name" : "Statuspage", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry for not catching this.....in the JSON file we alphabetize the sites by their name. Can you move this entry to the S section?
Changed it to alphabetical order.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changed it to be in alphabetical order.
wmn-data.json
Outdated
@@ -659,7 +659,7 @@ | |||
{ | |||
"name" : "Blogspot", | |||
"uri_check" : "http://{account}.blogspot.com", | |||
"strip_bad_char" : ".", | |||
rip_bad_char" : ".", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What happened with all these strip_bad_char
modifications?
Corrected the accidental changes.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Corrected the changes.
Contributed the configuration for statuspage.io.