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

System communication send test email #5814

Closed
2 tasks done
kevinkrugerCCV opened this issue Mar 29, 2024 · 3 comments
Closed
2 tasks done

System communication send test email #5814

kevinkrugerCCV opened this issue Mar 29, 2024 · 3 comments
Labels
Fixed in v16.5 Type: Bug Confirmed bugs or reports that are very likely to be bugs.

Comments

@kevinkrugerCCV
Copy link

Description

While using a system communication, if you use the "Send test" feature, the user selected will have their email changed to the email address used for the test and after the test is sent, the email does not change back as designed.

Actual Behavior

Testing a system communication requires you to select a target person but after selecting the "Send Test" and input a different email. Rock will update the target persons email to be the test email address.
image

Expected Behavior

After sending the test email, I would expect the target persons email to be reverted back to the appropriate address.

Steps to Reproduce

  • Go to "admin tools" menu
  • Click on "Communications"
  • Select "System Communications"
  • Select the "Preview" option on one of the system communications.
image - Select the "Send Test" on the right side of the communication. image - Use a different email than the one used for the target person - Visit target persons profile page to see email changed to that used for the test.

Issue Confirmation

  • Perform a search on the Github Issues to see if your bug or enhancement is already reported.
  • Try to reproduce the problem on a fresh install or on the demo site.

Rock Version

14.2

Client Culture Setting

en-US

@JimMichael
Copy link
Collaborator

I cannot duplicate this on 1.16.2 through prealpha.

@kevinkrugerCCV
Copy link
Author

Thank you @JimMichael,

Working with Michael at Triumph, he was able to recreate this on an instance of 16.2 and 16.3. I was just able to do this under 1.16.2.1

Just to confirm, this is happening in the system communication no the communication wizard. The target persons email address will change to the CurrentPerson's email.
a. Ted's email address is ted@rock.com
b. Logged in under admin@organization.com
c. Set Ted as the target person in the system communication preview page, send a test to "mytest@yahoo.com".
d. When sent, Ted's email address will get changed to "admin@organization.com"

image

@sparkdevnetwork-service sparkdevnetwork-service added Type: Bug Confirmed bugs or reports that are very likely to be bugs. Status: Confirmed It's clear what the subject of the issue is about, and what the resolution should be. labels Apr 1, 2024
@JimMichael
Copy link
Collaborator

Ah, the original steps didn’t specify you had to change the target person (it defaults to yourself) and just change the email of the (defaulted) target person (which works fine for me and doesn’t change the person’s actual email on the profile), so that must be the key.

@PraveenMathew92 PraveenMathew92 added Fixed in v16.5 and removed Status: Confirmed It's clear what the subject of the issue is about, and what the resolution should be. labels Apr 8, 2024
kylehenning pushed a commit that referenced this issue Apr 17, 2024
…ation Preview block failing to revert the test email back to the original email. (Fixes #5814)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fixed in v16.5 Type: Bug Confirmed bugs or reports that are very likely to be bugs.
Projects
None yet
Development

No branches or pull requests

4 participants