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: deprecate refreshClients methods #26956

Merged
merged 11 commits into from
Nov 22, 2022

Conversation

felipe-rod123
Copy link
Contributor

Proposed changes (including videos or screenshots)

Created the 'assets.refreshClients' endpoint for the apps/meteor/client/views/admin/settings/groups/AssetsGroupPage.tsx file.

Issue(s)

Steps to test or reproduce

Further comments

@codecov
Copy link

codecov bot commented Sep 27, 2022

Codecov Report

Merging #26956 (50bcb51) into develop (90fe2ed) will increase coverage by 1.53%.
The diff coverage is n/a.

❗ Current head 50bcb51 differs from pull request most recent head 771cfdf. Consider uploading reports for the commit 771cfdf to get more accurate results

Impacted file tree graph

@@             Coverage Diff             @@
##           develop   #26956      +/-   ##
===========================================
+ Coverage    40.48%   42.02%   +1.53%     
===========================================
  Files          840      814      -26     
  Lines        18329    17841     -488     
  Branches      2052     1988      -64     
===========================================
+ Hits          7421     7498      +77     
+ Misses       10622    10055     -567     
- Partials       286      288       +2     
Flag Coverage Δ
e2e 42.02% <ø> (+1.53%) ⬆️

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

@felipe-rod123 felipe-rod123 marked this pull request as ready for review September 27, 2022 21:03
@felipe-rod123 felipe-rod123 requested review from a team as code owners September 27, 2022 21:03
ggazzo
ggazzo previously approved these changes Sep 28, 2022
@ggazzo ggazzo added this to the 5.3.0 milestone Sep 28, 2022
Copy link
Member

@sampaiodiego sampaiodiego left a comment

Choose a reason for hiding this comment

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

@ggazzo should we remove this functionality completely? the button "apply and refresh clients" doesn't work at all.. no reason to keep it there.

@d-gubert d-gubert modified the milestones: 5.3.0, 5.4.0 Oct 19, 2022
@dionisio-bot dionisio-bot bot added stat: ready to merge PR tested and approved waiting for merge and removed stat: needs QA labels Nov 21, 2022
@ggazzo ggazzo changed the title Chore: create refreshClients endpoint Chore: deprecate refreshClients methods Nov 21, 2022
@kodiakhq kodiakhq bot merged commit 1bb9ee5 into develop Nov 22, 2022
@kodiakhq kodiakhq bot deleted the chore/create-refreshClients-endpoint branch November 22, 2022 00:43
MartinSchoeler pushed a commit that referenced this pull request Nov 28, 2022
Co-authored-by: Guilherme Gazzo <5263975+ggazzo@users.noreply.github.com>
MartinSchoeler pushed a commit that referenced this pull request Nov 28, 2022
Co-authored-by: Guilherme Gazzo <5263975+ggazzo@users.noreply.github.com>
@ggazzo ggazzo mentioned this pull request Dec 5, 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.

5 participants