Skip to content

fix: update urls in site cog#1743

Merged
mbaruh merged 2 commits into
mainfrom
dawnofmidnight/site-urls
Aug 22, 2021
Merged

fix: update urls in site cog#1743
mbaruh merged 2 commits into
mainfrom
dawnofmidnight/site-urls

Conversation

@whispersofthedawn
Copy link
Copy Markdown
Contributor

Some urls in the site cog are currently outdated and lead to the wrong pages. Some of them (like the Resources page) have redirects, while others (like the Asking Good Questions page, do not). This changes the constant to the site base url and updates the rest of the urls to be valid.

@wookie184
Copy link
Copy Markdown
Contributor

A github search for pythondiscord.com brings up another case of a redirect link (not in the site cog):

ASKING_GUIDE_URL = "https://pythondiscord.com/pages/asking-good-questions/"

There are also a few other locations where pythondiscord.com is used rather than the constant e.g.

NICKNAME_POLICY_URL = "https://pythondiscord.com/pages/rules/#nickname-policy"

Do you think these would be worth changing too?

@whispersofthedawn
Copy link
Copy Markdown
Contributor Author

The NICKNAME_POLICY_URL seems to be correct already, but I can fix the other.

@wookie184
Copy link
Copy Markdown
Contributor

wookie184 commented Aug 12, 2021

The NICKNAME_POLICY_URL seems to be correct already, but I can fix the other.

Yeah that one is the correct URL, just not using the site's name constants. There were a few others like that but I think the ASKING_GUIDE_URL is the only one with a redirect url. Just changing ASKING_GUIDE_URL is fine if you think the others are out of scope, replacing the text with constants isn't particularly important.

@MarkKoz MarkKoz added a: information Related to information commands: (doc, help, information, reddit, site, tags) p: 2 - normal Normal Priority t: bug Something isn't working labels Aug 13, 2021
Copy link
Copy Markdown
Contributor

@Bluenix2 Bluenix2 left a comment

Choose a reason for hiding this comment

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

Looks good, I see nothing wrong.

@Xithrius Xithrius requested a review from mbaruh August 22, 2021 09:34
@mbaruh mbaruh enabled auto-merge August 22, 2021 19:58
@mbaruh mbaruh merged commit 4a59465 into main Aug 22, 2021
@mbaruh mbaruh deleted the dawnofmidnight/site-urls branch August 22, 2021 19:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

a: information Related to information commands: (doc, help, information, reddit, site, tags) p: 2 - normal Normal Priority t: bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants