Skip to content

Fix notifying own activities - #565

Merged
nickvergessen merged 1 commit into
masterfrom
fix-notifying-own-activities
Mar 19, 2021
Merged

Fix notifying own activities#565
nickvergessen merged 1 commit into
masterfrom
fix-notifying-own-activities

Conversation

@tcitworld

Copy link
Copy Markdown
Member

$createStream was unused and the || true was odd. This should be correct behaviour as far as I understand it.

Comment threadlib/NotificationGenerator.php Outdated
$createStream was unused. This *should* be correct behaviour.
Signed-off-by: Thomas Citharel <tcit@tcit.fr>
@tcitworld
tcitworldforce-pushed the fix-notifying-own-activities branch from 74f2ff0 to ca76bc8CompareMarch 18, 2021 14:20

@icewind1991icewind1991 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

🙈

@nickvergessen
nickvergessen merged commit ee56a2b into masterMar 19, 2021
@delete-merged-branch
delete-merged-branchBot deleted the fix-notifying-own-activities branch March 19, 2021 15:46
@nickvergessen

Copy link
Copy Markdown
Member

/backport to stable21

@nickvergessen

Copy link
Copy Markdown
Member

/backport to stable20

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@tcitworld@nickvergessen@icewind1991