Skip to content

Conversation

@sampaiodiego
Copy link
Member

Proposed changes (including videos or screenshots)

Issue(s)

Steps to test or reproduce

Further comments

@ggazzo
Copy link
Member

ggazzo commented Jul 21, 2022

image

@sampaiodiego sampaiodiego marked this pull request as ready for review July 27, 2022 15:01
@sampaiodiego sampaiodiego changed the title [IMPROVE] Use single change stream to watch [IMPROVE] Use single change stream to watch DB changes Jul 27, 2022
@RocketChat RocketChat deleted a comment from lgtm-com bot Jul 27, 2022
@sampaiodiego sampaiodiego requested review from a team July 27, 2022 19:51
@RocketChat RocketChat deleted a comment from lgtm-com bot Jul 27, 2022
@ggazzo ggazzo added stat: ready to merge PR tested and approved waiting for merge and removed stat: needs QA labels Jul 28, 2022
@ggazzo ggazzo removed the request for review from a team July 28, 2022 19:26
@kodiakhq kodiakhq bot merged commit 9989223 into develop Jul 28, 2022
@kodiakhq kodiakhq bot deleted the improve-db-watcher branch July 28, 2022 19:27
@sampaiodiego sampaiodiego added this to the 5.0.1 milestone Jul 28, 2022
gabriellsh added a commit that referenced this pull request Jul 29, 2022
…ctor/locationModal

* 'develop' of github.com:RocketChat/Rocket.Chat:
  i18n: Makes the text less ambiguous (#20895)
  Chore: Missing some English translation keywords (#20131)
  Chore: Exclude private/public folders from typecheck (#26399)
  [NEW] Marketplace apps page new list view layout (#26181)
  Chore: Rewrite custom OAuth Modals to react (#26204)
  i18n: Manual sync from LingoHub (#26397)
  Chore: Options in BaseRaw model could possibly be undefined (#26395)
  [IMPROVE] Use single change stream to watch DB changes (#26336)
  [IMPROVE] Remove public and node_modules folders from TypeScript server watcher (#26391)
@murtaza98 murtaza98 mentioned this pull request Aug 2, 2022
gabriellsh added a commit that referenced this pull request Aug 2, 2022
…history

* 'develop' of github.com:RocketChat/Rocket.Chat:
  Chore: Cache playwright (#26432)
  [FIX] Empty results on `im.list` endpoint (#26438)
  [IMPROVE] Upgrade nivo and React Query (#26338)
  Chore: Upgrade Fuselage packages to next dist-tag (#26435)
  Chore: Omnichannel endpoints e2e tests (#26376)
  Chore: Refactor UserInfo to typescript (#26323)
  Chore: Rewrite Location modal to React (#26196)
  Chore: Rewrite SaveToWebdav Modal to React Component (#24365)
  [IMPROVE] validateParams to accept different validators per request method (#26357)
  Regression: Fix spacing problem on AppStatus component (#26421)
  Chore: Convert client/views/account/security folder to ts (#26413)
  i18n: Makes the text less ambiguous (#20895)
  Chore: Missing some English translation keywords (#20131)
  Chore: Exclude private/public folders from typecheck (#26399)
  [NEW] Marketplace apps page new list view layout (#26181)
  Chore: Rewrite custom OAuth Modals to react (#26204)
  i18n: Manual sync from LingoHub (#26397)
  Chore: Options in BaseRaw model could possibly be undefined (#26395)
  [IMPROVE] Use single change stream to watch DB changes (#26336)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

stat: ready to merge PR tested and approved waiting for merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants