Skip to content

Copy notifications from topic only for experts#182

Merged
ruKurz merged 1 commit intomasterfrom
hotfix/#180-create-request-broken
Dec 21, 2017
Merged

Copy notifications from topic only for experts#182
ruKurz merged 1 commit intomasterfrom
hotfix/#180-create-request-broken

Conversation

@mrsimpson
Copy link
Copy Markdown
Member

Fixes #180

Since normal users don't have a subscription for the expert topic, there was an exception raised when trying to copy the notifications from the parent room.

@ghost ghost assigned mrsimpson Dec 21, 2017
@ghost ghost added the progress:working label Dec 21, 2017
@mrsimpson mrsimpson requested a review from ruKurz December 21, 2017 13:47
Copy link
Copy Markdown

@ruKurz ruKurz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

straight forward

@ruKurz ruKurz merged commit 95518eb into master Dec 21, 2017
@ghost ghost removed the progress:working label Dec 21, 2017
@ruKurz ruKurz deleted the hotfix/#180-create-request-broken branch December 21, 2017 14:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants