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

membership#9 - show user a notification on back-office renewals #14088

Merged
merged 1 commit into from
Apr 21, 2019

Conversation

MegaphoneJon
Copy link
Contributor

@MegaphoneJon MegaphoneJon commented Apr 19, 2019

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

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

@civibot
Copy link

civibot bot commented Apr 19, 2019

(Standard links)

@civibot civibot bot added the master label Apr 19, 2019
@eileenmcnaughton
Copy link
Contributor

test this please

@eileenmcnaughton
Copy link
Contributor

looks good

@eileenmcnaughton eileenmcnaughton merged commit 7c9c7ef into civicrm:master Apr 21, 2019
@MegaphoneJon MegaphoneJon deleted the membership-9 branch July 10, 2019 16:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants