Skip to content
This repository has been archived by the owner on Jul 23, 2020. It is now read-only.

Include what was changed in notification email #1990

Open
aslakknutsen opened this issue Jan 23, 2018 · 2 comments
Open

Include what was changed in notification email #1990

aslakknutsen opened this issue Jan 23, 2018 · 2 comments

Comments

@aslakknutsen
Copy link
Collaborator

aslakknutsen commented Jan 23, 2018

For WorkItem Update and Comment Update notifications, only part of the new state is included. In the Work Item Update case the email is a notification that 'something changed', but with no real information to what.

The current UX can be found here: https://redhat.invisionapp.com/share/XHD0O8A5B#/248448905_Notification-Comment-Updated

Related

@surajssd
Copy link
Collaborator

  • Now WIT sends revisionID along with the workitemId to notification service. send revisionid to notification service fabric8-services/fabric8-wit#2176

  • What notification service can now do is parse this revisionID from the custom attributes and in order to generate workitem update template it should query the WIT endpoint GET /workitems/{wiID}/events. And in the output you will get all the events and here notification service should filter on the basis of revisionID sent earlier by WIT. This particular revision info can help generate updated template.

@joshuawilson
Copy link
Member

How much of this is left to do? I know some improvements have been made.
cc @alexeykazakov

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants