search

MissVeronica / um-admin-user-profile-update-email

Extension to Ultimate Member with an email template for sending an email to the site admin when an UM User Profile is updated either by the User or an Admin
GNU Affero General Public License v3.0
1 stars 3 forks source link

Email sending before changed fields are saved? #3

Open HmCody opened 1 year ago

HmCody commented 1 year ago

I was trying to troubleshoot something I had messed up with including meta fields in the body of the emails (too many curly brackets), when I ran into a different problem.

It seems as if the email is being sent before the changes are committed to the database. If I change a field and save, the email does not include the change, but if I then update a different field and save again, the first field is shown correctly but the second is not.

Let me know if you need any additional information.

Thanks!

MissVeronica commented 1 year ago

It seems as if the email is being sent before the changes are committed to the database.

No, the email is sent before the redirect after updating the database and UM cache is updated by the plugin. I can't reproduce your issue. Which field did you update?

HmCody commented 1 year ago

Some of my meta fields. I just checked again and it only happens when I am doing the update on the backend with the user-edit form. The front end does seem to work correctly.

On Tue, Oct 17, 2023 at 1:05 PM MissVeronica @.***> wrote:

It seems as if the email is being sent before the changes are committed to the database.

No, the email is sent before the redirect after updating the database and UM cache is updated by the plugin. I can't reproduce your issue. Which field did you update?

— Reply to this email directly, view it on GitHub https://github.com/MissVeronica/um-admin-user-profile-update-email/issues/3#issuecomment-1766827906, or unsubscribe https://github.com/notifications/unsubscribe-auth/AH6IOW4437JEW4FIQOGUVC3X723HFAVCNFSM6AAAAAA6DXC7YWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONRWHAZDOOJQGY . You are receiving this because you authored the thread.Message ID: <MissVeronica/um-admin-user-profile-update-email/issues/3/1766827906@ github.com>

MissVeronica commented 1 year ago

I have posted an updated plugin. Changes are about backend updating.

HmCody commented 1 year ago

Perfect! Thanks for fixing it.

On Wed, Oct 18, 2023 at 5:31 PM MissVeronica @.***> wrote:

I have posted an updated plugin. Changes are about backend updating.

— Reply to this email directly, view it on GitHub https://github.com/MissVeronica/um-admin-user-profile-update-email/issues/3#issuecomment-1769353182, or unsubscribe https://github.com/notifications/unsubscribe-auth/AH6IOWZD7I3CDV75JYJOA53YABDCRAVCNFSM6AAAAAA6DXC7YWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONRZGM2TGMJYGI . You are receiving this because you authored the thread.Message ID: <MissVeronica/um-admin-user-profile-update-email/issues/3/1769353182@ github.com>

MissVeronica commented 1 year ago

Thanks for your feedback. Today I have made some additional UX changes in version 4.1.0