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

Change the blog applications use of querystrings to filter blogs to use segments in the URL path #10

Closed
rjsparks opened this issue Sep 5, 2019 · 0 comments

Comments

@rjsparks
Copy link
Member

rjsparks commented Sep 5, 2019

The current scheme using URLs like /blog/whatever?primary_key=3&secondary_key=27 defeats CDN caching and the ability for editors to use wagtail's tools for linking to internal pages.

Instead, the app's use of the URL space should be modified to something similar to:
/blog/news/ (filtering to news entries)
/blog/<other secondary topic>/ (filtering to the set of secondary topics)
/blog/entry/whatever

The notion of PrimaryTopic can go away (news can be moved to a SecondaryTopic, which can just become Topic). Topics will need slugs.

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

No branches or pull requests

1 participant