-
Notifications
You must be signed in to change notification settings - Fork 13.8k
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
chore(explore): make adhoc columns available without UX BETA ff #17921
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🚀
Codecov Report
@@ Coverage Diff @@
## master #17921 +/- ##
=======================================
Coverage 67.07% 67.07%
=======================================
Files 1609 1609
Lines 64899 64896 -3
Branches 6866 6864 -2
=======================================
- Hits 43533 43532 -1
+ Misses 19500 19499 -1
+ Partials 1866 1865 -1
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
/testenv up |
@jinghua-qa Ephemeral environment creation is currently limited to committers. |
/testenv up |
@rusackas Ephemeral environment spinning up at http://54.213.1.12:8080. Credentials are |
I am not able to see adhoc columns feature in groupby in the test env, do i need to enable dnd and dnd hybrid FF? Screen.Recording.2022-01-04.at.11.03.54.AM.mov |
Yes, DnD is a requirement, sorry for not putting that information in PR description. Let me update |
/testenv up FEATURE_ENABLE_EXPLORE_DRAG_AND_DROP=true FEATURE_ENABLE_DND_WITH_CLICK_UX=true |
@jinghua-qa Ephemeral environment spinning up at http://54.218.150.117:8080. Credentials are |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Ephemeral environment shutdown and build artifacts deleted. |
…he#17921) * chore(explore): make adhoc columns available without UX BETA ff * Remove redundant imports
…he#17921) * chore(explore): make adhoc columns available without UX BETA ff * Remove redundant imports
…he#17921) * chore(explore): make adhoc columns available without UX BETA ff * Remove redundant imports
SUMMARY
The feature introduced in #17379 was available only with UX BETA feature flag enabled. Since we received positive feedback from the users who tried the feature out, it's time to make it available to everyone!
This PR removes the dependency on UX BETA feature flag. To learn more about the feature, check out the linked PR
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
TESTING INSTRUCTIONS
ADDITIONAL INFORMATION
CC @rusackas @jinghua-qa @rosemarie-chiu