Skip to content

Conversation

@Sg312
Copy link
Contributor

@Sg312 Sg312 commented Dec 23, 2025

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)

@vercel
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

@greptile-apps
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

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.

2 participants