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

[FIX] Notify private settings changes even on public settings changed #13369

Merged
merged 1 commit into from
Feb 5, 2019

Conversation

tassoevan
Copy link
Contributor

@tassoevan tassoevan commented Feb 5, 2019

Closes #11604
Closes #12399
Closes #12144
An else directive introduced the issue at version 0.67.0: https://github.com/RocketChat/Rocket.Chat/pull/10749/files#diff-ff8f9f02a53856a978c533000a92bd12R70

Co-authored-by: Guilherme Gazzo <guilherme.gazzo@rocket.chat>
@ggazzo
Copy link
Member

ggazzo commented Feb 5, 2019

I don't know how you found this bug, well done :)

@ggazzo ggazzo merged commit 6080c8a into develop Feb 5, 2019
@ggazzo ggazzo deleted the fix/settings-persistence branch February 5, 2019 22:02
@johnyb0y
Copy link

johnyb0y commented Feb 5, 2019

Thanks!

@tassoevan tassoevan modified the milestones: 0.75.0, 0.74.3 Feb 6, 2019
@sampaiodiego sampaiodiego mentioned this pull request Feb 14, 2019
wreiske added a commit to wreiske/Rocket.Chat that referenced this pull request Apr 28, 2019
…nto ldap-admin-groups

* 'develop' of https://github.com/RocketChat/Rocket.Chat: (21 commits)
  Regression: Active room was not being marked (RocketChat#14276)
  Rename Cloud to Connectivity Services & split Apps in Apps and Marketplace (RocketChat#14211)
  LingoHub based on develop (RocketChat#14178)
  [IMPROVE] Replace livechat inquiry dialog with preview room (RocketChat#13986)
  Bump version to 0.74.3
  Room loading improvements (RocketChat#13471)
  [FIX] Invalid condition on getting next livechat agent over REST API endpoint (RocketChat#13360)
  [IMPROVE] Open rooms quicker (RocketChat#13417)
  [FIX] "Test Desktop Notifications" not triggering a notification (RocketChat#13457)
  [FIX] Translated and incorrect i18n variables (RocketChat#13463)
  Regression: Remove console.log on email translations (RocketChat#13456)
  [FIX] Properly escape custom emoji names for pattern matching (RocketChat#13408)
  [FIX] Not translated emails (RocketChat#13452)
  Added missing package dependency (RocketChat#13437)
  Update Russian localization (RocketChat#13244)
  [IMPROVE] Allow configure Prometheus port per process via Env Var (RocketChat#13436)
  [IMPROVE] Add API option "permissionsRequired" (RocketChat#13430)
  [FIX] Several Problems on HipChat Importer (RocketChat#13336)
  Add the missing uniqueId to the push notifications (RocketChat#13423)
  [FIX] Notify private settings changes even on public settings changed (RocketChat#13369)
  ...
Sign up for free to 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.

4 participants