Skip to content

fix: Remove upload and download buttons form the graph-engine toolbar#713

Closed
not-in-stock wants to merge 1 commit intomasterfrom
andrei/2911-fix-graph-upload
Closed

fix: Remove upload and download buttons form the graph-engine toolbar#713
not-in-stock wants to merge 1 commit intomasterfrom
andrei/2911-fix-graph-upload

Conversation

@not-in-stock
Copy link
Copy Markdown

Description

Fixes a part of #2911

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How to test this

  1. Open graph engine canvas.
  2. A toolbar shouldn't contain upload and download buttons.

Screenshots or video (if necessary):


  • This PR's content is either hand-written or thoroughly code-reviewed by me and I fully understand and am accountable for this PR.

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Jun 30, 2025

⚠️ No Changeset found

Latest commit: cca80bd

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@not-in-stock not-in-stock force-pushed the andrei/2911-fix-graph-upload branch from 32db134 to c4109c0 Compare June 30, 2025 13:13
@not-in-stock not-in-stock force-pushed the andrei/2911-fix-graph-upload branch from c4109c0 to cca80bd Compare June 30, 2025 13:18
@LilithWittmann
Copy link
Copy Markdown
Contributor

You cant just remove it globally from all places the graph engine is used. You need to make that configurable.

@not-in-stock not-in-stock deleted the andrei/2911-fix-graph-upload branch June 30, 2025 14:21
@not-in-stock
Copy link
Copy Markdown
Author

not-in-stock commented Jun 30, 2025

You cant just remove it globally from all places the graph engine is used. You need to make that configurable.

@LilithWittmann you're right it can be configured in the Editor component.
Like this https://github.com/tokens-studio/studio-app/pull/2962/commits/7e8bcdc5beb4903aa3a30c7488375bd78caf6134

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