I went to add "Yes" to the RAN and TTA columns of the index page's complaint table, when I decided it would be much easier if the Complaint model itself knew whether it had any RAN / TTA activity. Then I realized heck, it would be great if it knew if it was a new complaint, and if it could return its status already formatted and then I kinda just kept redoing things....
moves formatted status behavior out of helpers, simplifies views
adds methods for new?, has_monitoring_review?, and has_tta_report?
In terms of the RAN / TTA issue, this is what I've got:
Acceptance Criteria
are these helpful and logical changes?
nothing got broken along the way (like status formatting / sorting)
RAN and TTA have "Yes" if the issue is linked to something
How to test
Start the app, navigate to a complaint page and add a TTA and / or RAN link. Now on the complaints page, you should see "Yes" in the appropriate column.
Description of change
I went to add "Yes" to the RAN and TTA columns of the index page's complaint table, when I decided it would be much easier if the Complaint model itself knew whether it had any RAN / TTA activity. Then I realized heck, it would be great if it knew if it was a new complaint, and if it could return its status already formatted and then I kinda just kept redoing things....
new?,has_monitoring_review?, andhas_tta_report?In terms of the RAN / TTA issue, this is what I've got:
Acceptance Criteria
How to test
Start the app, navigate to a complaint page and add a TTA and / or RAN link. Now on the complaints page, you should see "Yes" in the appropriate column.
Issue(s)
Checklist
To be completed by the submitter:
To the Reviewer
This project is using Conventional Comments to give structure and context to PR comments. Please use these labels in your comments.