-
Notifications
You must be signed in to change notification settings - Fork 1
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
Support claims index page filters #447
Conversation
17edfe8
to
9a9c825
Compare
467c2e9
to
57ffe46
Compare
Review app track and pay deployed to https://track-and-pay-447.test.teacherservices.cloud was deleted |
app/javascript/controllers/claims_support_filter_search_controller.js
Outdated
Show resolved
Hide resolved
This is also a pretty complex change. It would help reviewers if this PR was broken down into step-by-step commits. |
57ffe46
to
fe1286b
Compare
fe1286b
to
46fd3ed
Compare
46fd3ed
to
9f93aed
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Very nice!
We need to allow our support users to filter the claims. Two filters, School and Provider filters have 2 search fields where you can search by school name or provider to get the filter you want. This is done by javascript in a stimulus controller. The other 2 filters, Submitted after and Submitted before filters are date filters and use the govuk_date_field form helper. You cannot set the value of this helper as it's sending 3 params not just 1. So to persist the value of these filters we need a form object. Claims::Support::Claims::FilterForm is a form object that is meant to just house the params and all the logic that these filter need. All the params of the filters are in the url.
9f93aed
to
6de4acb
Compare
Context
We need to allow our support users to filter the claims.
Two filters,
School
andProvider
filters have 2 search fields whereyou can search by school name or provider to get the filter you want.
This is done by javascript in a stimulus controller.
The other 2 filters,
Submitted after
andSubmitted before
filters aredate filters and use the
govuk_date_field
form helper. You cannot setthe value of this helper as it's sending 3 params not just 1. So to
persist the value of these filters we need a form object.
Claims::Support::Claims::FilterForm
is a form object that is meant tojust house the params and all the logic that these filters need.
All the params of the filters are in the url.
Changes proposed in this pull request
Claims::Support::Claims::FilterForm
Stimulus controller
Styling changes
Guidance to review
Sign in as colin
Go to support claims index
Filter claims by all the filters
Use the search too
Screenshots
Peek.2024-04-11.16-10.mp4