-
Notifications
You must be signed in to change notification settings - Fork 216
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
fix(monitoring): refresh monitoring on content changes #1151
Conversation
user = flask.g.get('user', {}) | ||
push_notification( | ||
'content:update', | ||
user=str(user.get('_id', '')), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we use common.py.get_user()
here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yep I just didn't want to import from archive here, so I will move the func here and import it into archive
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
actually I will move it into apps.auth
, imo it should be there
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah makes sense
👍 |
9ab4766
to
ec69a3c
Compare
👍 |
1 similar comment
👍 |
ec69a3c
to
e336d93
Compare
👍 Rebase needed |
e336d93
to
633584c
Compare
there is single event `content:update` with info about item, desk and stage which client can use to figure out what to update in monitoring. SD-3204 SD-3213
633584c
to
4ebc858
Compare
fix(monitoring): refresh monitoring on content changes
there is single event
content:update
with info about item, deskand stage which client can use to figure out what to update in monitoring.
SD-3204 SD-3213