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: Remove Meteor.call in favor of Meteor.callAsync - 1 #28576

Merged
merged 11 commits into from
Mar 23, 2023

Conversation

KevLehman
Copy link
Contributor

@KevLehman KevLehman commented Mar 23, 2023

ARCH-568
ARCH-569
ARCH-571
ARCH-572
ARCH-573
ARCH-574
ARCH-575
ARCH-576
ARCH-577
ARCH-578
ARCH-579

Proposed changes (including videos or screenshots)

Issue(s)

Steps to test or reproduce

Further comments

@codecov
Copy link

codecov bot commented Mar 23, 2023

Codecov Report

Merging #28576 (91e576b) into develop (7dd7186) will decrease coverage by 0.03%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop   #28576      +/-   ##
===========================================
- Coverage    45.04%   45.02%   -0.03%     
===========================================
  Files          753      753              
  Lines        14662    14662              
  Branches      2129     2129              
===========================================
- Hits          6605     6602       -3     
- Misses        7755     7759       +4     
+ Partials       302      301       -1     
Flag Coverage Δ
e2e 44.98% <ø> (-0.03%) ⬇️

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

@KevLehman KevLehman marked this pull request as ready for review March 23, 2023 21:51
@rodrigok rodrigok added stat: ready to merge PR tested and approved waiting for merge stat: QA skipped and removed stat: needs QA labels Mar 23, 2023
@ggazzo ggazzo merged commit d3904b3 into develop Mar 23, 2023
@ggazzo ggazzo deleted the refactor/callAsync-1 branch March 23, 2023 22:39
gabriellsh added a commit that referenced this pull request Mar 24, 2023
…nto fix/avatarEmoji

* 'fix/avatarEmoji' of github.com:RocketChat/Rocket.Chat: (72 commits)
  refactor: `LivechatRooms` removal - 3 (#28506)
  fix: Livechat MessageList not auto scrolling on new message (#28547)
  chore: Add types for `Meteor.userAsync` (#28597)
  refactor: Easy replacements of Meteor.call to Meteor.callAsync (#28595)
  refactor: `Subscriptions` model - 2x (#28535)
  refactor: `LivechatRooms` 2 (#28504)
  regression: ldap sync not working (#28583)
  fix: Generic Table Search (#28401)
  refactor: Remove `Apps persistence` model (#28510)
  refactor(models): Use Messages Raw model (5/N) (#28590)
  refactor: federation promise.await methods 2/N (#28593)
  refactor: federation Promise.await 1/N (#28591)
  feat: Make the marketplace search bar placeholder dynamic (#28394)
  fix: Bring unseen app requests badge back and minor refactors (#28348)
  refactor: Remove `apps` & `app logs` models (#28509)
  refactor: Convert meteor call to callAsync (2/N) (#28588)
  refactor: Remove `Meteor.call` in favor of `Meteor.callAsync` - 1 (#28576)
  refactor: Restore removed function for counting on LivechatDepartmentAgents (#28577)
  refactor: Convert meteor call to callAsync (#28575)
  refactor: Remove optional return of room coordinator (#28592)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stat: QA skipped stat: ready to merge PR tested and approved waiting for merge
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants