Skip to content

Fix wiki URLs in README.md - #152

Merged
un33k merged 3 commits into
un33k:cifrom
sagesharp:patch-1
Nov 20, 2025
Merged

Fix wiki URLs in README.md#152
un33k merged 3 commits into
un33k:cifrom
sagesharp:patch-1

Conversation

@sagesharp

Copy link
Copy Markdown

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

sagesharp commented Sep 18, 2024

Copy link
Copy Markdown
Author

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

un33k commented Sep 24, 2024

Copy link
Copy Markdown
Owner

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

@kurtmckee

Copy link
Copy Markdown
Contributor

@un33k This may have slipped through the cracks. I'm pinging you because it appears to still be relevant.

@un33k
un33k merged commit a243eae into un33k:ci Nov 20, 2025
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.

3 participants