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

refactor: promise.await removal 8/N #28560

Merged
merged 62 commits into from
Mar 22, 2023
Merged

refactor: promise.await removal 8/N #28560

merged 62 commits into from
Mar 22, 2023

Conversation

ggazzo
Copy link
Member

@ggazzo ggazzo commented Mar 22, 2023

Proposed changes (including videos or screenshots)

Issue(s)

Steps to test or reproduce

Further comments

Base automatically changed from refactor/promise-07 to develop March 22, 2023 22:32
@ggazzo ggazzo marked this pull request as ready for review March 22, 2023 22:34
@ggazzo ggazzo requested review from a team as code owners March 22, 2023 22:34
@codecov
Copy link

codecov bot commented Mar 22, 2023

Codecov Report

Merging #28560 (0b1bd04) into develop (7675c1d) will decrease coverage by 0.03%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop   #28560      +/-   ##
===========================================
- Coverage    45.04%   45.02%   -0.03%     
===========================================
  Files          755      755              
  Lines        14622    14622              
  Branches      2108     2108              
===========================================
- Hits          6587     6583       -4     
- Misses        7737     7741       +4     
  Partials       298      298              
Flag Coverage Δ
e2e 44.98% <ø> (-0.03%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

@ggazzo ggazzo merged commit 69ad59d into develop Mar 22, 2023
@ggazzo ggazzo deleted the refactor/promise-08 branch March 22, 2023 23:03
gabriellsh added a commit that referenced this pull request Mar 23, 2023
…ove/full-name-username

* 'develop' of github.com:RocketChat/Rocket.Chat: (54 commits)
  chore: ignore `.eslintcache` (#28475)
  refactor: convert omnichannel callbacks to ts (#28564)
  refactor: Allow RoomCoordinator.roomFind to be async (#28566)
  refactor(models): Use Messages Raw model (4/N) (#28558)
  refactor(client): `meteor/session` usage (1/N) (#28565)
  refactor: Don't use meteor collections on migrations (#28563)
  feat: VideoConference Guest mode and Conference Router (#28186)
  refactor: promise.await removal 8/N (#28560)
  refactor: checkUsernameAvailability to async (#28557)
  refactor(promise.await): Low hanging fruits (2/2) (#28556)
  fix: Changing the app's error verification (#28450)
  refactor(promise.await): Low hanging fruits (1/N) (#28555)
  fix: Thread message preview (#28454)
  refactor: remove Promise.await from LDAP files (#28527)
  refactor: canAccessRoomId to async (#28540)
  refactor: Remove `LivechatInquiry` model (#28487)
  refactor(models): Use Messages Raw model (3/N) (#28549)
  refactor: remove Promise.await (#28539)
  chore: Make CI fail if checks fail (#28552)
  refactor: remove Promise.await from oauth manager (#28530)
  ...
gabriellsh added a commit that referenced this pull request Mar 23, 2023
…ixSearch

* 'develop' of github.com:RocketChat/Rocket.Chat: (141 commits)
  ci(docker): use correct var name
  fix: Prevent blank space on live chat form validations (#28243)
  ci(docker): Push Docker tag for commit hash (#28578)
  fix: marketplace doc link wrong redirect (#28466)
  refactor: move `Subscriptions` 1x (#28531)
  chore: ignore `.eslintcache` (#28475)
  refactor: convert omnichannel callbacks to ts (#28564)
  refactor: Allow RoomCoordinator.roomFind to be async (#28566)
  refactor(models): Use Messages Raw model (4/N) (#28558)
  refactor(client): `meteor/session` usage (1/N) (#28565)
  refactor: Don't use meteor collections on migrations (#28563)
  feat: VideoConference Guest mode and Conference Router (#28186)
  refactor: promise.await removal 8/N (#28560)
  refactor: checkUsernameAvailability to async (#28557)
  refactor(promise.await): Low hanging fruits (2/2) (#28556)
  fix: Changing the app's error verification (#28450)
  refactor(promise.await): Low hanging fruits (1/N) (#28555)
  fix: Thread message preview (#28454)
  refactor: remove Promise.await from LDAP files (#28527)
  refactor: canAccessRoomId to async (#28540)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants