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

Fix anchor link for agent uninstall #19937

Merged
merged 7 commits into from
Jun 25, 2024
Merged

Fix anchor link for agent uninstall #19937

merged 7 commits into from
Jun 25, 2024

Conversation

zwass
Copy link
Member

@zwass zwass commented Jun 21, 2024

For #19936

Checklist for submitter

If some of the following don't apply, delete the relevant line.

  • Changes file added for user-visible changes in changes/, orbit/changes/ or ee/fleetd-chrome/changes.
    See Changes files for more information.
  • Manual QA for all new/changed functionality

@zwass zwass requested a review from a team as a code owner June 21, 2024 20:41
RachelElysia
RachelElysia previously approved these changes Jun 21, 2024
Copy link
Member

@rachaelshaw rachaelshaw left a comment

Choose a reason for hiding this comment

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

@zwass mind adding a redirect here and linking to it from the modal, instead of to the docs directly? (That'll make it quicker to fix if docs are shuffled around again in the future.)

I'd suggest:

'GET /learn-more-about/uninstalling-fleetd': '/docs/using-fleet/faq#how-can-i-uninstall-fleetd',

@zwass
Copy link
Member Author

zwass commented Jun 21, 2024

@rachaelshaw I used the approach you described.

rachaelshaw
rachaelshaw previously approved these changes Jun 21, 2024
Copy link

codecov bot commented Jun 21, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 52.05%. Comparing base (688d86a) to head (172197d).
Report is 46 commits behind head on main.

Current head 172197d differs from pull request most recent head 0617f99

Please upload reports for the commit 0617f99 to get more accurate results.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #19937      +/-   ##
==========================================
- Coverage   53.61%   52.05%   -1.57%     
==========================================
  Files         430      415      -15     
  Lines       10633     9713     -920     
  Branches     3208     3134      -74     
==========================================
- Hits         5701     5056     -645     
+ Misses       4909     4635     -274     
+ Partials       23       22       -1     
Flag Coverage Δ
fleetd-chrome ?
frontend 52.05% <ø> (-0.03%) ⬇️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

rachaelshaw
rachaelshaw previously approved these changes Jun 21, 2024
@zwass
Copy link
Member Author

zwass commented Jun 25, 2024

@rachaelshaw can you please review again after formatting fix?

@zwass zwass merged commit b2dec6d into main Jun 25, 2024
11 checks passed
@zwass zwass deleted the zwass-patch-3 branch June 25, 2024 20:06
georgekarrv pushed a commit that referenced this pull request Jul 1, 2024
For #19936

# Checklist for submitter

If some of the following don't apply, delete the relevant line.

<!-- Note that API documentation changes are now addressed by the
product design team. -->

- [x] Changes file added for user-visible changes in `changes/`,
`orbit/changes/` or `ee/fleetd-chrome/changes`.
See [Changes
files](https://fleetdm.com/docs/contributing/committing-changes#changes-files)
for more information.
- [ ] Manual QA for all new/changed functionality
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants