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

Bug: Adding an order (via the +order icon when you hover on a members' name) to an existing members' account fires the new member checkout email #49

Open
MaryOJob opened this issue May 31, 2022 · 1 comment · May be fixed by #54

Comments

@MaryOJob
Copy link

I can replicate this consistently both on local and live test sites.
To replicate:

  • Install the Add Member from Admin Add-On
  • Go to the member's list table
  • Hover the mouse on any member and click the +order link
  • Add an order
  • See email inbox for checkout emails sent to both admin and the member

The expected behavior should be that an admin is able to add an order to a member's account without getting a new member's email.

Note that when an order is added via the orders page > Orders > Add New Order - no email templates are sent to either the admin or the member as this invoice can be emailed to the member by clicking the "Email" link.

PMPro Version 2.8.3
PMPro Add Member from Admin A 0.7.1
WordPress 6.0
Memberlite Theme 4.5.2
PHP 7.4.29

@MaryOJob MaryOJob changed the title Adding an order (via the +order icon when you hover on a members' name) to an existing members' account fires the new member checkout email Bug: Adding an order (via the +order icon when you hover on a members' name) to an existing members' account fires the new member checkout email Jun 1, 2022
@andrewlimaza
Copy link
Collaborator

The reason this happens is that the +order is actually changing the member's level. To confirm this, you can add an order for a different level ID which then changes their level.

@andrewlimaza andrewlimaza linked a pull request Jan 3, 2023 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants