Skip to content
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

fix: show popover on paste #10970

Merged
merged 3 commits into from
Aug 5, 2024
Merged

fix: show popover on paste #10970

merged 3 commits into from
Aug 5, 2024

Conversation

almeidabbm
Copy link
Contributor

@almeidabbm almeidabbm commented Aug 2, 2024

Description:

When pasting a value into a filter our default behavior was to always add it as multiple values when the clipboard contents had commas.
This PR makes it so that when pasting a comma separated value we ask for user input on how the value should be treated

  • Shows a popover when pasting to filters if the clipboard values has comma separated values

chart filters

Screen.Recording.2024-08-02.at.09.37.34.mov

dashboard filters
Screenshot 2024-08-02 at 09 41 50

Reviewer actions

  • I have manually tested the changes in the preview environment
  • I have reviewed the code
  • I understand that "request changes" will block this PR from merging

@almeidabbm almeidabbm self-assigned this Aug 2, 2024
@owlas owlas requested a deployment to fix/show-popover-on-paste - jaffle_db_pg_13 PR #10970 August 2, 2024 08:40 — with Render Abandoned
@owlas owlas deployed to fix/show-popover-on-paste - headless-browser PR #10970 August 2, 2024 08:40 — with Render Active
Copy link

netlify bot commented Aug 2, 2024

Deploy Preview for peaceful-bassi-cbf284 canceled.

Name Link
🔨 Latest commit 3fe2b0d
🔍 Latest deploy log https://app.netlify.com/sites/peaceful-bassi-cbf284/deploys/66ad06ad691e8500083a3b96

@almeidabbm almeidabbm removed the request for review from TuringLovesDeathMetal August 2, 2024 08:41
@owlas owlas temporarily deployed to fix/show-popover-on-paste - lightdash PR #10970 August 2, 2024 08:41 — with Render Destroyed
@owlas owlas deployed to fix/show-popover-on-paste - headless-browser PR #10970 August 2, 2024 08:45 — with Render Active
Copy link
Collaborator

@rephus rephus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This works great ! nice UI

@almeidabbm
Copy link
Contributor Author

This works great ! nice UI

Just waiting on: #10989 to be merged so we can run CI and make sure it isn't breaking anything 💪

@owlas owlas temporarily deployed to fix/show-popover-on-paste - lightdash PR #10970 August 2, 2024 16:17 — with Render Destroyed
@owlas owlas temporarily deployed to fix/show-popover-on-paste - headless-browser PR #10970 August 2, 2024 16:17 — with Render Destroyed
@almeidabbm almeidabbm merged commit 811aebc into main Aug 5, 2024
47 of 52 checks passed
@almeidabbm almeidabbm deleted the fix/show-popover-on-paste branch August 5, 2024 08:04
lightdash-bot pushed a commit that referenced this pull request Aug 5, 2024
## [0.1202.5](0.1202.4...0.1202.5) (2024-08-05)

### Bug Fixes

* show popover on paste ([#10970](#10970)) ([811aebc](811aebc))
@lightdash-bot
Copy link
Collaborator

🎉 This PR is included in version 0.1202.5 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

4 participants