Skip to content

chore: add default app icon#71

Merged
zimeg merged 4 commits into
mainfrom
add-app-icon
May 15, 2026
Merged

chore: add default app icon#71
zimeg merged 4 commits into
mainfrom
add-app-icon

Conversation

@srtaalej
Copy link
Copy Markdown
Contributor

@srtaalej srtaalej commented Apr 27, 2026

Summary

  • Adds a default app icon at assets/icon.png
  • The Slack CLI auto-detects this path and uploads it when running locally

Test plan

  • Verify the icon renders correctly in the PR diff
  • Run slack run --experiment set-icon and confirm the icon appears on the app

@srtaalej srtaalej requested a review from a team as a code owner April 27, 2026 14:27
@srtaalej srtaalej added the enhancement New feature or request label May 7, 2026
@srtaalej srtaalej self-assigned this May 7, 2026
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.

@srtaalej For this monorepo we ought use nested directories I think for?

  • claude-agent-sdk/assets/icon.png
  • openai-agents-sdk/assets/icon.png
  • pydantic-ai/assets/icon.png

@srtaalej srtaalej requested a review from zimeg May 15, 2026 17:28
@slack-samples
Copy link
Copy Markdown
Contributor

slack-samples Bot commented May 15, 2026

CLI create commands for each template in this PR:

slack create -t slack-samples/bolt-python-support-agent --subdir claude-agent-sdk --branch add-app-icon
slack create -t slack-samples/bolt-python-support-agent --subdir openai-agents-sdk --branch add-app-icon
slack create -t slack-samples/bolt-python-support-agent --subdir pydantic-ai --branch add-app-icon

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.

@srtaalej Let's get this merged I added pydantic-ai too 🚢 💨

@zimeg zimeg merged commit 357b581 into main May 15, 2026
8 checks passed
@zimeg zimeg deleted the add-app-icon branch May 15, 2026 20:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants