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

[#7785] Make request-related URLs member routes #8058

Merged
merged 3 commits into from
Jan 9, 2024
Merged

Conversation

alexander-griffen
Copy link
Contributor

@alexander-griffen alexander-griffen commented Dec 18, 2023

Relevant issue(s)

Fixes #7785

What does this do?

This makes all routes related to a request have the :url_title be after the request part of the URL.

Why was this needed?

Reduce permutations of URLs

@alexander-griffen alexander-griffen self-assigned this Dec 18, 2023
Change the order of parts of relevant urls, and fixing the specs this
breaks
@gbp gbp force-pushed the 7785-member-routes branch from 203af5a to bc6412c Compare January 8, 2024 13:19
@gbp gbp changed the title [#7785] WIP: Make request-related URLs member routes [#7785] Make request-related URLs member routes Jan 8, 2024
@gbp gbp marked this pull request as ready for review January 8, 2024 13:24
@gbp gbp self-requested a review January 8, 2024 13:24
@gbp gbp force-pushed the 7785-member-routes branch from bc6412c to f2ed33b Compare January 8, 2024 14:16
@gbp gbp merged commit fe50c2c into develop Jan 9, 2024
7 checks passed
gbp added a commit that referenced this pull request Jan 9, 2024
These got left out of #8058 somehow.
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.

Make all request-related URL actions a member route
2 participants