Skip to content
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

Merged
merged 4 commits into from
Nov 25, 2024
Merged

Update wmn-data.json #906

merged 4 commits into from
Nov 25, 2024

Conversation

0x9404
Copy link
Contributor

@0x9404 0x9404 commented Nov 24, 2024

Contributed the configuration for statuspage.io.

Copy link
Owner

@WebBreacher WebBreacher left a 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 Show resolved Hide resolved
wmn-data.json Outdated
"name" : "Statuspage",
"uri_check" : "https://{account}.statuspage.io/api/v2/status.json",
"e_code" : 200,
"e_string" : "id",
Copy link
Owner

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"],
Copy link
Owner

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

@0x9404 0x9404 left a 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",
Copy link
Owner

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

@0x9404 0x9404 left a 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" : ".",
Copy link
Owner

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

@0x9404 0x9404 left a comment

Choose a reason for hiding this comment

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

Corrected the changes.

@WebBreacher WebBreacher merged commit 9aa7448 into WebBreacher:main Nov 25, 2024
2 checks passed
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.

2 participants