Skip to content

Fix wiki URLs in README.md #152

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

Open
wants to merge 3 commits into
base: ci
Choose a base branch
from
Open

Fix wiki URLs in README.md #152

wants to merge 3 commits into from

Conversation

sagesharp
Copy link

See commit description.

un33k and others added 3 commits March 1, 2024 13:19
On Feb 4, the name of the top-level wiki page was changed from "Python Slugify Wiki" to "Welcome to the python-slugify wiki!".

That change to the wiki page title changed all the URLs for the wiki pages. If you tried to click a wiki URL in the README.md, GitHub would redirect you to a form to create a new wiki page with that name.

Fix this by updating the URLs in README.md to match the new wiki top page title.

TODO: I don't know if there are other wiki URLs in other documentation files. The instances in README.md were the ones I have time to fix while quickly trying to use python-slugify for a personal project. Thanks for all your work maintaining this project!
@sagesharp
Copy link
Author

sagesharp commented Sep 18, 2024

Ah, it seems the instructions on the wiki for sending in a pull request are incorrect? The instructions say to open a PR against the CI branch. However, the CI branch seems to be missing some commits from the main branch, so this GitHub PR automatically tried to add them. I'm not as familiar with GitHub process flow (most of my projects I have commit rights to), so please have patience with me. I'll double check I didn't skip any steps in the instructions.

@un33k
Copy link
Owner

un33k commented Sep 24, 2024

Thank you for this PR. It will be pulled in on the next update cycle, along with at least one hot issue.

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