Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

fix(integrations): hide spotify tool from toolbar #2544

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

Open
Sg312 wants to merge 1 commit into main
base: main
Choose a base branch
Loading
from fix/hide-spotify

Conversation

@Sg312
Copy link
Contributor

@Sg312 Sg312 commented Dec 23, 2025
edited
Loading

Summary

Hide spotify from toolbar

Type of Change

  • Bug fix

Testing

Manual

Checklist

  • Code follows project style guidelines
  • Self-reviewed my changes
  • Tests added/updated and passing
  • No new warnings introduced
  • I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

greptile-apps[bot] reacted with thumbs up emoji
Copy link

vercel bot commented Dec 23, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Review Updated (UTC)
docs Skipped Skipped Dec 23, 2025 6:50am

Copy link
Contributor

greptile-apps bot commented Dec 23, 2025

Greptile Summary

Added hideFromToolbar: true configuration property to the Spotify block to hide it from the toolbar UI.

  • Single line addition at line 15, following the established pattern used by other blocks like api_trigger, webhook, and manual_trigger
  • Change is properly placed after category: 'tools' and before bgColor, maintaining consistent property ordering
  • All Spotify functionality (OAuth, operations, subBlocks) remains intact and accessible through other means
  • No breaking changes or logic modifications

Confidence Score: 5/5

  • This PR is completely safe to merge with zero risk.
  • The change is a minimal, single-line configuration addition that follows an established pattern used across 9 other blocks in the codebase. The property is boolean, non-breaking, and only affects UI visibility without touching any logic, functionality, or data handling.
  • No files require special attention

Important Files Changed

Filename Overview
apps/sim/blocks/blocks/spotify.ts Added hideFromToolbar: true to hide Spotify from toolbar. Simple configuration change with no logical issues.

Sequence Diagram

sequenceDiagram
 participant Dev as Developer
 participant Config as SpotifyBlock Config
 participant Toolbar as Toolbar Component
 participant API as Spotify Integration
 
 Dev->>Config: Add hideFromToolbar: true
 Note over Config: Property added at line 15<br/>after category: 'tools'
 
 Toolbar->>Config: Read block configuration
 Config-->>Toolbar: hideFromToolbar: true
 Note over Toolbar: Spotify not rendered<br/>in toolbar UI
 
 Note over API: All Spotify operations<br/>remain accessible<br/>via other means
 
 Note over Dev,API: Block hidden from UI<br/>but functionality preserved
Loading

Base automatically changed from staging to main December 27, 2025 05:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

AltStyle によって変換されたページ (->オリジナル) /