add route "Generate Court Report" to sidebar#992
Conversation
…select current_user.casa_cases with 2 buttons
…s all assigned case
erik-trantt
changed the title
| <div class="card-body"> | ||
| <h5 class="card-title"><strong>Generate Court Report</strong></h5> | ||
| <p class="card-text"> | ||
| The Court Report is pre-filled with information of your case. You can select among currently active cases assigned to you. The document is in Microsoft Word format (.dox). |
There was a problem hiding this comment.
Is .dox correct? I would expect .doc or .docx but I might be out of touch with windows
"information of your case" sounds funny - maybe "information for your case" instead?
There was a problem hiding this comment.
@compwron You're right. It's a typo. The correct type is .docx.
I'll also change the wordings to "information for your case".
|
Looking good! I see that this is still in "draft" state so I won't review it yet :) |
|
The commit was in CI build queue last night, so I waited till it finished to "open" the PR. Is it okay to open while CI build hasnt finished yet?
|
erik-trantt
added
the
zzz_archived: 🧐 Status: Ready for Review
@erik-trantt it is ok to open the PR without a green build on this repo. :) |
|
@compwron Thanks for the clarification 👍 |
erik-trantt
added
hacktoberfest-accepted
3 participants
What github issue is this PR for, if any?
Resolves #923
What changed, and why?
<domain_name>/case_court_reports/, and placed it to the sidebar.How will this affect user permissions?
N/A
Currently, all users can see visit the link and see the form.
How is this tested? (please write tests!) 💖💪
spec/views/layouts/sidebar.html.erb_spec.rbapp/views/case_court_reports/index.html.erbScreenshots please :)