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

Prevent +order from changing members level #54

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

andrewlimaza
Copy link
Collaborator

@andrewlimaza andrewlimaza commented Jan 3, 2023

  • BUG FIX/ENHANCEMENT: Prevent +order logic from changing the user's membership level.

Resolves: #49

* BUG FIX/ENHANCEMENT: Prevent +order logic from changing the user's membership level.
@andrewlimaza
Copy link
Collaborator Author

andrewlimaza commented Jan 3, 2023

I think it'd be nice to have a field option when in the +order view that, if selected it will still allow you to change the user's level during order creation to keep backwards compatibility.

@patric-boehner
Copy link

Thank you for this update. This solves a headache we were experiencing with adding mailed-in check payments and triggering a level change that was reported as a cancelation to reports, which caused a lot of confusion for the client.

I know the +order feature doesn't support membership expiration fields, but It would be great to see that happen at some point in the future if possible. It would save a lot of back and for when we add in batches of check payments at a time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants