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

Chore: Convert current-chats to useQuery #26931

Merged
merged 4 commits into from
Sep 23, 2022
Merged

Conversation

ggazzo
Copy link
Member

@ggazzo ggazzo commented Sep 23, 2022

Proposed changes (including videos or screenshots)

Issue(s)

Steps to test or reproduce

Further comments

@ggazzo ggazzo requested a review from a team as a code owner September 23, 2022 03:46
@ggazzo ggazzo added this to the 5.2.0 milestone Sep 23, 2022
@ggazzo ggazzo force-pushed the chore/current-chats-query branch from b478a88 to 093b0e8 Compare September 23, 2022 04:08
@codecov
Copy link

codecov bot commented Sep 23, 2022

Codecov Report

Merging #26931 (effb319) into develop (cfd738a) will decrease coverage by 0.01%.
The diff coverage is 74.12%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop   #26931      +/-   ##
===========================================
- Coverage    40.84%   40.82%   -0.02%     
===========================================
  Files          793      800       +7     
  Lines        17848    17912      +64     
  Branches      1941     1959      +18     
===========================================
+ Hits          7290     7313      +23     
- Misses       10258    10303      +45     
+ Partials       300      296       -4     
Flag Coverage Δ
e2e 40.82% <74.12%> (-0.02%) ⬇️

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

MartinSchoeler
MartinSchoeler previously approved these changes Sep 23, 2022
@MartinSchoeler MartinSchoeler dismissed tassoevan’s stale review September 23, 2022 15:11

Stale by his own changes :)

@ggazzo ggazzo added the stat: ready to merge PR tested and approved waiting for merge label Sep 23, 2022
@kodiakhq kodiakhq bot merged commit 0beec2f into develop Sep 23, 2022
@kodiakhq kodiakhq bot deleted the chore/current-chats-query branch September 23, 2022 18:36
@tassoevan tassoevan mentioned this pull request Oct 13, 2022
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
None yet
Development

Successfully merging this pull request may close these issues.

3 participants