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: Rooms Model 5/N #28619

Merged
merged 124 commits into from
Mar 26, 2023
Merged

refactor: Rooms Model 5/N #28619

merged 124 commits into from
Mar 26, 2023

Conversation

ggazzo
Copy link
Member

@ggazzo ggazzo commented Mar 25, 2023

Proposed changes (including videos or screenshots)

Issue(s)

Steps to test or reproduce

Further comments

Base automatically changed from rooms-to-async-4 to develop March 25, 2023 22:47
@ggazzo ggazzo requested a review from rodrigok March 25, 2023 23:12
@ggazzo ggazzo marked this pull request as draft March 25, 2023 23:47
rodrigok
rodrigok previously approved these changes Mar 26, 2023
@codecov
Copy link

codecov bot commented Mar 26, 2023

Codecov Report

Merging #28619 (9c31863) into develop (bf1cb19) will increase coverage by 0.08%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop   #28619      +/-   ##
===========================================
+ Coverage    44.91%   44.99%   +0.08%     
===========================================
  Files          753      753              
  Lines        14665    14665              
  Branches      2130     2130              
===========================================
+ Hits          6587     6599      +12     
+ Misses        7776     7764      -12     
  Partials       302      302              
Flag Coverage Δ
e2e 44.95% <ø> (+0.08%) ⬆️

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

@rodrigok rodrigok marked this pull request as ready for review March 26, 2023 19:07
@rodrigok rodrigok merged commit faa7c52 into develop Mar 26, 2023
@rodrigok rodrigok deleted the rooms-to-async-5 branch March 26, 2023 19:08
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