Skip to content

docs: fix broken link to removing an app pages#232

Merged
haleychaas merged 1 commit intomainfrom
docs-haley-fix-broken-link
Sep 25, 2025
Merged

docs: fix broken link to removing an app pages#232
haleychaas merged 1 commit intomainfrom
docs-haley-fix-broken-link

Conversation

@haleychaas
Copy link
Copy Markdown
Contributor

Summary

Fixes a reported broken link for uninstalling an app.

Requirements

@haleychaas haleychaas requested a review from a team as a code owner September 24, 2025 20:48
@codecov
Copy link
Copy Markdown

codecov Bot commented Sep 24, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 62.99%. Comparing base (7ec1470) to head (74afc84).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #232   +/-   ##
=======================================
  Coverage   62.99%   62.99%           
=======================================
  Files         212      212           
  Lines       21857    21857           
=======================================
  Hits        13769    13769           
+ Misses       7022     7021    -1     
- Partials     1066     1067    +1     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Copy Markdown
Contributor

@lukegalbraithrussell lukegalbraithrussell left a comment

Choose a reason for hiding this comment

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

 ____  _____ ____  _____ _____ ____ _____ 
/  __\/  __//  __\/    //  __//   _Y__ __\
|  \/||  \  |  \/||  __\|  \  |  /   / \  
|  __/|  /_ |    /| |   |  /_ |  \_  | |  
\_/   \____\\_/\_\\_/   \____\\____/ \_/  
                                          

@zimeg zimeg changed the title Docs: Fix broken link docs: fix broken link to removing an app pages Sep 25, 2025
Copy link
Copy Markdown
Member

@zimeg zimeg left a comment

Choose a reason for hiding this comment

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

@haleychaas LGTM! I'm a big fan of these PRs when discovered 🙏 ✨

I left a comment about links overall but nothing to stop this from merging IMHO! 🚢

All good things come to an end! If you need to uninstall the Slack CLI, run the commands below. Note that these instructions will uninstall the Slack CLI, but not its dependencies. Follow [these instructions](https://docs.deno.com/runtime/manual/tools/script_installer#uninstall) to uninstall Deno.

✨ **Just need to uninstall an app?** Refer to [uninstall an app from your workspace](/tools/deno-slack-sdk/guides/creating-an-app#uninstall-app).
✨ **Just need to uninstall an app?** Refer to [Removing an app](/tools/deno-slack-sdk/guides/removing-an-app).
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

⭐ praise: Thanks so much for the fast update!

🔗 https://docs.slack.dev/tools/deno-slack-sdk/guides/removing-an-app

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👁️‍🗨️ thought: I'm somewhat curious also if we want to move this page to the CLI sidebar instead in a follow up PR?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

yes, that's a good idea! I'll add it to our backlog and handle it separately

@zimeg zimeg added the docs M-T: Documentation work only label Sep 25, 2025
@zimeg zimeg added this to the Next Release milestone Sep 25, 2025
@haleychaas haleychaas merged commit 5ba77b0 into main Sep 25, 2025
6 checks passed
@haleychaas haleychaas deleted the docs-haley-fix-broken-link branch September 25, 2025 13:39
@mwbrooks mwbrooks added the semver:patch Use on pull requests to describe the release version increment label Oct 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs M-T: Documentation work only semver:patch Use on pull requests to describe the release version increment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants