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

[#1663] Add additional routes for /help/house_rules #1671

Closed
wants to merge 1 commit into from

Conversation

mdeuk
Copy link
Collaborator

@mdeuk mdeuk commented May 2, 2023

Relevant issue(s)

#1663

What does this do?

This adds a new route which allows for the /help/house_rules page to be accessed using different paths.

Why was this needed?

Chat on #1663, and predecessors, has raised the issue of language.

Allowing more possibilities allows us to ensure that messages to users are easy to understand, as "house rules" may not mean something to many.

Implementation notes

The basic implementation isn't anything especially complex - it is a redirect to /help/house_rules.

We match :help_house_rules_resolve against a constraint. The following are accepted values:

  • conditions_of_use
  • site_rules
  • terms_and_conditions
  • terms_of_service
  • terms_of_use
  • the_legal_stuff
  • the_rules

This doesn't change the existing route for /help/house_rules, as whilst it is trivial to include that here, it is a bit less elegant 1.

Screenshots

N/A

Notes to reviewer

I've tested this on a sandbox install (running 0.43.0.0 / rails 7.0.4.3 / ruby 3.2.1) and didn't encounter any problems.

Footnotes

  1. not that this is elegant… but, it prevents a slight nuisance in regard to how we render the sidebar (e.g. not rendering a clickable 'house rules' link when we don't have to).

@mdeuk mdeuk marked this pull request as ready for review May 2, 2023 07:24
@mdeuk
Copy link
Collaborator Author

mdeuk commented May 2, 2023

@garethrees
Copy link
Member

Thanks for this, but I think we should instead resolve #1663 and then update the URL. If we start using a multitude of URLs it might get quite confusing (e.g. if we send a user a URL for "conditions of use" and then they try to find the page through browsing, they won't find it.) We'd also then have to consider whether to maintain these URLs over the long term, which while small, adds a difficult-to-resolve issue and a little more code complexity.

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