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

Article List CBV Use #4298

Open
2 tasks
ajrbyers opened this issue Jun 26, 2024 · 1 comment
Open
2 tasks

Article List CBV Use #4298

ajrbyers opened this issue Jun 26, 2024 · 1 comment
Labels
dev-ready This issue has been refined and is ready for development. t-shirt XXL Extra extra large

Comments

@ajrbyers
Copy link
Member

It has been noted that a few views are not currently using the Article list CBV pattern.

  • Search
  • Funder Articles

I've added this so we can discuss it in triage.

@joemull
Copy link
Member

joemull commented Jul 4, 2024

We want to combine the search/ and articles/ endpoints and forms so that the user experience is seamless and they only have one set of order by, relevance, etc.

We are thinking of:

  • making full-text search a facet in the class-based view
  • preserving enough of the current functionality (namely the view) to support people who have built something on top of it
  • Some sorting options are dependent on context: Relevance is only available on search and will need to be considered. We discussed making relevance the default option when the user actively navigates to the search page (clicks on search button or interacts with a search bar)

@mauromsl mauromsl added t-shirt XXL Extra extra large dev-ready This issue has been refined and is ready for development. labels Jul 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dev-ready This issue has been refined and is ready for development. t-shirt XXL Extra extra large
Projects
None yet
Development

No branches or pull requests

3 participants