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

Added the "slug_mode" option to URL generation. #156

Closed
wants to merge 1 commit into from

Conversation

cosmochefe
Copy link

This commit helps with localization of URLs for sites that use category and tag names that have accented characters. By using the "slug_mode" option in the site's configuration file, the user can apply the correspondent mode of the "Utils.slugify" method to the generated directory name and URL. Issue #121 should be resolved by this.

@ashmaroli
Copy link
Member

@cosmochefe Thank you for the contribution.
I'm on board with the proposal. But this needs a test to see if a scenario similar to #121 is resolved and stays so, with some test coverage.

@ashmaroli
Copy link
Member

Related: #100

@ashmaroli
Copy link
Member

Prior art: #129

@cosmochefe
Copy link
Author

cosmochefe commented Apr 12, 2020 via email

@ashmaroli
Copy link
Member

I'm fine Alvaro, thank you. Hope you're doing great as well.
Thanks for letting me know this is is your first GitHub contribution ever. I'll provide you with some orientation:

  • Ideally, a GitHub project would have a CONTRIBUTING.md or similar document that explains what to do when contributing to a project. But not every project has that document.
  • In a situation where the project repository doesn't have strict guidelines, you just submit a pull request and wait for the project's maintainer(s) to leave a review and then proceed to do as requested in the review.
  • It is considered good practice to:
    • propose related changes yet narrow down the scope as much as possible.
    • have commit message(s) that are of 50 characters or lesser and best describe the contained changes.
    • open the pull request with an appropriate title and description
    • if the repository has a pull request template, always best to follow along with it.
    • accompany at least one test related to the proposed change so that everyone can instantly gauge what the expected behavior is. Additionally, tests ensure that a future change doesn't break your expectations.
    • where requested, accompany appropriate documentation so that users know how to employ the feature you have proposed.

Regarding your proposal, unfortunately, there's already a prior submission (#129). You can get an idea about testing by viewing that PR — basically add a post with accented characters in the title and assert that the end-result is what you expect to get.

I've left a message to the original author to revisit their submission. If that PR gets dropped, this pull request will get picked up. If it doesn't, well.. this could be an exercise for you....

@ashmaroli
Copy link
Member

Superseded by prior submission #129

@ashmaroli ashmaroli closed this Apr 29, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants