You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@superset.apache.org by "Mayankaggarwal8055 (via GitHub)" <gi...@apache.org> on 2026/05/02 13:20:05 UTC

Re: [PR] fix(explore): explain disabled chart overwrite option [superset]

Mayankaggarwal8055 commented on PR #39796:
URL: https://github.com/apache/superset/pull/39796#issuecomment-4363865274

   Hey @yeaight7, this is a nice UX improvement 👍
   
   The message makes sense and definitely helps explain why the overwrite option is disabled. One small thought — do you think it would help to also hint at the alternative (like saving as a new chart)? Might make it a bit more actionable for users.
   
   Also just checking — is this string covered by translation like the rest of the UI?
   
   Overall looks clean and well scoped. Nice work!


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org