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

Improve test run change notification email #199

Closed
okainov opened this Issue Feb 1, 2018 · 1 comment

Comments

Projects
None yet
2 participants
@okainov
Copy link
Contributor

okainov commented Feb 1, 2018

Right now there are email notifications being sent after test run change, but they don't really contain important info what exactly was updated.

Test run 8 has been updated for you.

Links

Test run: XXXrun/8/

Basic run information

Summary: XXX
Managed: XXX.
Default tester: XXX.
Product: XXX
Product version: XXX
Build: XXX
Estimated time: 0:00:00
Notes:

Test plan information

Test Plan: XXX

Actual change was in setting "Finished" to True and this is not displayed at all in the email. I would expect having only changes in email, but not all test run details

@atodorov

This comment has been minimized.

Copy link
Member

atodorov commented Feb 18, 2018

See my comment on #216. Unfortunately we don't have the internal machinery to easily make the distinction what has been changed. I'll keep researching on this topic since it is a good feature request.

@atodorov atodorov self-assigned this Feb 18, 2018

atodorov added a commit that referenced this issue Jul 20, 2018

Show actual changes in post_update email notifications. Fix #199
history of changes is now recorded as a unified diff and we simply
send that via email

@atodorov atodorov closed this in eb0f80b Jul 22, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.
You signed in with another tab or window. Reload to refresh your session. You signed out in another tab or window. Reload to refresh your session.