Skip to content

fix: open the list of apps if no known apps are found for "app settings" command#314

Merged
zimeg merged 4 commits intomainfrom
zimeg-fix-app-settings-blank
Jan 29, 2026
Merged

fix: open the list of apps if no known apps are found for "app settings" command#314
zimeg merged 4 commits intomainfrom
zimeg-fix-app-settings-blank

Conversation

@zimeg
Copy link
Member

@zimeg zimeg commented Jan 28, 2026 •
edited by mwbrooks
Loading

Changelog

slack app settings will now open the App Settings Home Page (https://api.slack.com/apps) when the command is run outside of a project. This allows you to see your complete list of apps.

Summary

This PR opens the app settings dashboard if the app settings command is run either outside of a project or if no apps exist.

Preview

$ slack app settings

🏠 App Settings
   https://api.slack.com/apps

Reviewers

Navigate to the pages of app settings with these commands:

$ slack app settings                              # api.slack.com/apps
$ slack app settings --apihost=dev1234.slack.com  # api.dev1234.slack.com/apps
$ slack create asdf
$ cd asdf
$ slack app settings                              # api.slack.com/apps
$ slack install
$ slack app settings                              # api.slack.com/apps/A0123456789

Requirements

@zimeg zimeg added this to the Next Release milestone Jan 28, 2026
@zimeg zimeg self-assigned this Jan 28, 2026
@zimeg zimeg requested a review from a team as a code owner January 28, 2026 23:30
@zimeg zimeg added bug M-T: confirmed bug report. Issues are confirmed when the reproduction steps are documented semver:patch Use on pull requests to describe the release version increment labels Jan 28, 2026
@codecov
Copy link

codecov bot commented Jan 28, 2026 •
edited
Loading

Codecov Report

❌ Patch coverage is 89.47368% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 64.87%. Comparing base (8321475) to head (e0482a9).
⚠️ Report is 4 commits behind head on main.

Files with missing lines Patch % Lines
cmd/app/settings.go 89.47% 1 Missing and 1 partial ⚠️
Additional details and impacted files 
@@            Coverage Diff             @@
##             main     #314      +/-   ##
==========================================
+ Coverage   64.85%   64.87%   +0.02%     
==========================================
  Files         212      212              
  Lines       17538    17555      +17     
==========================================
+ Hits        11374    11389      +15     
- Misses       5094     5097       +3     
+ Partials     1070     1069       -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.

mwbrooks
mwbrooks approved these changes Jan 29, 2026
View reviewed changes
Copy link
Member

@mwbrooks mwbrooks left a comment

Choose a reason for hiding this comment

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

❤️ Love it! While an error is the natural first choice in this situation, I like the default of opening all app settings instead.

🧪 Thanks for adding tests!

@zimeg
Copy link
Member Author

zimeg commented Jan 29, 2026

@mwbrooks Thanks so much for the review! I'm excited for connections between surfaces all the more so let's now merge this! 👾 ✨

@zimeg zimeg merged commit 358f1cc into main Jan 29, 2026
8 checks passed
@zimeg zimeg deleted the zimeg-fix-app-settings-blank branch January 29, 2026 00:30
@zimeg zimeg mentioned this pull request Jan 30, 2026
Merged
2 tasks
@mwbrooks mwbrooks added the changelog Use on updates to be included in the release notes label Jan 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mwbrooks mwbrooks mwbrooks approved these changes

@hello-ashleyintech hello-ashleyintech Awaiting requested review from hello-ashleyintech

Assignees

@zimeg zimeg

Labels

bug M-T: confirmed bug report. Issues are confirmed when the reproduction steps are documented changelog Use on updates to be included in the release notes semver:patch Use on pull requests to describe the release version increment

Projects

None yet

Milestone

v3.12.0

Development

Successfully merging this pull request may close these issues.

2 participants

@zimeg @mwbrooks

Comments