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

(WIP) dev/core#1331 Don't freeze fields for auto-renew memberships #16322

Closed
wants to merge 3 commits into from

Conversation

wmortada
Copy link
Contributor

@wmortada wmortada commented Jan 17, 2020

Overview

If a membership is set to auto-renew some of the fields are frozen, which means that a user is unable to edit them when they visit the edit membership screen.

This PR changes this behaviour so that these fields are frozen only if the auto-renew can be cancelled. If the membership is auto-renew but can't be cancelled the fields aren't frozen and can be edited with an appropriate warning.

See https://lab.civicrm.org/dev/core/issues/1331 for more details and discussion.

Before

If the membership is set to auto-renew, the following fields are frozen (whether or not the auto-renew can be cancelled):

  • Membership organisation / Membership type
  • Membership status

image

After

If the membership is set to auto-renew and it is possible to cancel the auto-renewal the fields are frozen (no change).

image

If the membership is set to auto-renew and it is not possible to cancel the auto-renewal the fields are not frozen and the user has the ability to override them if required. Help text with an appropriate warning is displayed.

image

Technical Details

None

Comments

The end date field was unfrozen in #15505. See https://lab.civicrm.org/dev/core/issues/1126 for details.

@civibot
Copy link

civibot bot commented Jan 17, 2020

(Standard links)

@civibot civibot bot added the master label Jan 17, 2020
@wmortada
Copy link
Contributor Author

wmortada commented Jan 17, 2020

The test failed with an error:

During parsing an error has been reported.

The error itself is:

No report files were found. Configuration error?

Not sure what that means.

@wmortada wmortada requested a review from mattwire January 17, 2020 16:09
@demeritcowboy
Copy link
Contributor

@wmortada You need to rebase the .tpl. If you click on Console Output at the left when viewing the test result you'll see the error.

@wmortada
Copy link
Contributor Author

@demeritcowboy thanks. I'll take a look.

@wmortada wmortada changed the title dev/core#1331 Don't freeze fields for auto-renew memberships (WIP) dev/core#1331 Don't freeze fields for auto-renew memberships Jan 31, 2020
@wmortada
Copy link
Contributor Author

Closing in favour of #16609

@wmortada wmortada closed this Feb 21, 2020
@wmortada wmortada deleted the core#1331-2 branch December 5, 2023 11:28
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