membership#9 - show user a notification on back-office renewals #14088
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Overview
When renewing a membership via the back-office form, no on-screen notification appears. This is technically a regression but appears to be quite old, so no need for urgency.
Before
No notification appears.
After
Notification appears.
![selection_791](https://user-images.githubusercontent.com/1796012/52441750-88ab7880-2aef-11e9-8a7f-35333b20c353.png)
Technical Details
The only functional change is removing the two
return
statements and removing the sentence about the end date (per Eileen's request to align with PR #12583). I additionally extracted the renewal message into its own method so that I could extend the tests to cover it.Comments
The screenshot above is from before I removed the sentence about the end date. This is also a redo of PR #13553.
https://lab.civicrm.org/dev/membership/issues/9