justinpark opened a new pull request, #34559:
URL: https://github.com/apache/superset/pull/34559

   ### SUMMARY
   
   In #31421 the existing saved query link ID has been replaced with a new 
permalink API. This change is currently impacting Airbnb services that 
reference the previous saved query links.
   
   To address this, in 5.0, we restored the original query link functionality 
to a "share link" action, while keeping the updated functionality for "copy 
query URL."
   Additionally, please note that a saved query ID always reflects the latest 
information when a saved query is edited. In contrast, the "copy query URL" 
uses the permalink method, which means it does not capture any changes made to 
the query after the URL is generated. To clarify this behavior, this commit 
updates the label to "copy current query URL."
   
   ### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
   
   After:
   
   <img width="274" height="252" alt="Screenshot 2025-08-05 at 10 10 16 AM" 
src="https://github.com/user-attachments/assets/80070aee-0c99-4c28-a868-245649699073";
 />
   
   <img width="266" height="253" alt="Screenshot 2025-08-05 at 10 10 20 AM" 
src="https://github.com/user-attachments/assets/0ff1a857-14cd-49da-9a86-512b45eb9810";
 />
   
   
   ### TESTING INSTRUCTIONS
   
   Go to Saved queries and then check the actions
   
   ### ADDITIONAL INFORMATION
   <!--- Check any relevant boxes with "x" -->
   <!--- HINT: Include "Fixes #nnn" if you are fixing an existing issue -->
   - [ ] Has associated issue:
   - [ ] Required feature flags:
   - [ ] Changes UI
   - [ ] Includes DB Migration (follow approval process in 
[SIP-59](https://github.com/apache/superset/issues/13351))
     - [ ] Migration is atomic, supports rollback & is backwards-compatible
     - [ ] Confirm DB migration upgrade and downgrade tested
     - [ ] Runtime estimates and downtime expectations provided
   - [ ] Introduces new feature or API
   - [ ] Removes existing feature or API
   


-- 
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-unsubscr...@superset.apache.org

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


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

Reply via email to