Closed PaulMorris closed 5 years ago
This version adds both "Edit" and "Renew" links (the current functionality) to the Enrollments > Approved table, for any applications where these features will currently work. (The ones that have a profile that shows up on the My Profile page.) See the first application in this screenshot:
This is a first step, and is ready for review, no longer WIP. (Ultimately we want this functionality to work for all applications in the list, and also for other pending and denied statuses.)
Tested by clicking the new links and confirming that they work as expected, taking the user to the same pages they can reach via My Profile page.
Latest version adds links for editing pending applications. The edits succeed, but the submitted date does not change, so it's 'edit in place' rather than 'edit and resubmit'. Not sure we want to merge yet given this behavior.
Also, the changes show both 'edit' and 'view' links when both are possible (all pending and some approved).
Aside: I tried using the same url for editing denied applications as with pending. The edit process succeeds but the status of the application remains "denied", so again an 'edit in place' rather than 'edit and resubmit'.
I rebased this off of my plumbing work and so this shouldn't land until PRs #997 and #993 do.
I think this is ready to go. Take one last look @PaulMorris ?
Since I started this PR I'm not allowed to give an 'approve' review, but I've reviewed the code and done manual testing. That turned up issue #1105, but that is not caused by this PR. (I added some tests for that issue in PR #1106.)
It's possible the new UI in this PR (for these new features) will surface similar issues in some cases. That will become clearer once issue #1105 is fixed. So we could either hold off on merging this PR until #1105 is fixed, to test further. Or we could go ahead and merge this PR (since the changes it makes are themselves good to go), and then deal with any further issues that arise in follow-up PRs.
@PaulMorris I have removed the enrollments section of the My Profile page. Please give it a look :)
@frankduncan I've added commits with tests for the server errors I found. (@ignore
d for the moment). Over to you, when you have a chance to take a look.
@PaulMorris Ah ha, it was due to the information getting lost as the provider travelled to and from xml. Thanks for the super useful tests, and I think it's working now :)
@frankduncan Looks good! Glad it was an easy fix. I'll go ahead and merge now.
WIP toward adding existing functionality (available on the "My Profile" page) to the table on the 'Enrollments > Approved' page.
Mixed results so far:
0
and so the link took you to the My Profile page with the message "Please select the profile to be updated."These errors make sense given that the only profile/application listed on 'My Profile' was the one that worked.
Issue #949 Give providers the ability to "edit and resubmit" applications