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

CRIMAPP-89 add search filters and attributes for return reasons report #185

Merged
merged 1 commit into from
Nov 24, 2023

Conversation

timpeat
Copy link
Member

@timpeat timpeat commented Nov 23, 2023

Description of change

Add office_code, provider_name, return_reason, and return details to search results.
Add reviewed_at filters
Allow sorting of search results by return_reason, office_code, and reference.

Link to relevant ticket

CRIMAPP-89

Notes for reviewer / how to test

Required by ministryofjustice/laa-review-criminal-legal-aid#465

@timpeat timpeat force-pushed the CRIMAPP-89-returned-reasons-report branch from 1a962d1 to 1cb20d6 Compare November 23, 2023 13:48
@timpeat timpeat changed the title oi CRIMAPP-89 add search filters and attributes for return reasons report Nov 23, 2023
@timpeat timpeat force-pushed the CRIMAPP-89-returned-reasons-report branch from 1cb20d6 to 6aabcc7 Compare November 23, 2023 13:56
@timpeat timpeat force-pushed the CRIMAPP-89-returned-reasons-report branch from 6aabcc7 to 602185f Compare November 24, 2023 13:37
@timpeat timpeat marked this pull request as ready for review November 24, 2023 13:37
@timpeat timpeat requested a review from a team as a code owner November 24, 2023 13:37
Copy link
Contributor

@hiboabd hiboabd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉

@timpeat timpeat merged commit 735fc18 into main Nov 24, 2023
4 checks passed
@timpeat timpeat deleted the CRIMAPP-89-returned-reasons-report branch November 24, 2023 14:59
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