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

Regression: Fix improper usage of useEndpointData #28050

Merged
merged 11 commits into from Feb 24, 2023

Conversation

KevLehman
Copy link
Contributor

Proposed changes (including videos or screenshots)

Issue(s)

Steps to test or reproduce

Further comments

OC-627

@KevLehman KevLehman requested a review from a team as a code owner February 15, 2023 16:14
@KevLehman KevLehman changed the title Fix improper usage of useEndpointData Regression: Fix improper usage of useEndpointData Feb 15, 2023
Copy link
Contributor

@MartinSchoeler MartinSchoeler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe we could update this to use tanstack's useQuery?

As we have deprecated useEndpointData

@renatobecker renatobecker added this to the 6.0.0 milestone Feb 16, 2023
@codecov
Copy link

codecov bot commented Feb 17, 2023

Codecov Report

Merging #28050 (d895d3e) into develop (5f6868d) will decrease coverage by 12.65%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##           develop   #28050       +/-   ##
============================================
- Coverage    44.85%   32.20%   -12.65%     
============================================
  Files          766      628      -138     
  Lines        14903    12485     -2418     
  Branches      2070     1849      -221     
============================================
- Hits          6684     4021     -2663     
- Misses        7925     8263      +338     
+ Partials       294      201       -93     
Flag Coverage Δ
e2e 32.20% <ø> (-12.61%) ⬇️

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

@murtaza98
Copy link
Contributor

Thanks for the review! Converted useEndpointData to tanstack's query. Pls check :)

MartinSchoeler
MartinSchoeler previously approved these changes Feb 17, 2023
Copy link
Contributor

@MartinSchoeler MartinSchoeler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just pushed some changes, FYI when you use invalidateQuery you don't need an effect since the query client has no direct dependency on the used useQuery

@dionisio-bot dionisio-bot bot added stat: ready to merge PR tested and approved waiting for merge and removed stat: needs QA labels Feb 23, 2023
@ggazzo ggazzo merged commit 5e8f807 into develop Feb 24, 2023
@ggazzo ggazzo deleted the regression/chats-panel-missing-permission branch February 24, 2023 00:41
gabriellsh added a commit that referenced this pull request Feb 24, 2023
…jumpToMessage

* 'develop' of github.com:RocketChat/Rocket.Chat:
  Regression: Fix RegisterForm Translations (#28156)
  Chore: e2e omnichannel tests - Close every opened page after the test (#28152)
  Chore: Remove fail-fast from matrix strategy (#28153)
  Chore: Fix houston metadata
  Regression: Fix improper usage of useEndpointData (#28050)
  Regression: View mode preferences without the selected option by default (#28120)
  Chore: Fix fetching Apps-Engine and MongoDB versions for release notes (#28129)
  Regression: Slash commands detected at any line (#28142)
  Regression: App requests notify admin function generating wrong link (#28111)
  Chore: Improve time of meteor build (#28128)
  Chore: Update tokens (#28079)
  Regression: Fix sidebar horizontal scrolling (#28125)
  Chore: Presence cap tweaks (#28058)
  Chore: Fix Services Docker Build
  Chore: Custom CSS  (#27993)
  Chore: Fix CI Docker Alpine build missing (#28138)
  Chore: Fix CI docker publish (#28134)
  Chore: Refactor CI to improve code reusability and organization (#28108)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants