Skip to content

fix(explore): restored hidden field values has discarded#29349

Merged
justinpark merged 2 commits into
apache:masterfrom
justinpark:hotfix--hidden-control-value-discarded
Jun 24, 2024
Merged

fix(explore): restored hidden field values has discarded#29349
justinpark merged 2 commits into
apache:masterfrom
justinpark:hotfix--hidden-control-value-discarded

Conversation

@justinpark
Copy link
Copy Markdown
Member

@justinpark justinpark commented Jun 24, 2024

SUMMARY

When the x-axis was changed back to a time item from a non-time control item, the values of hidden fields were not properly restored to their original state, causing them to be consistently recognized as hidden items and preventing form_data from being sent.
This commit fixes the issue and includes test code to prevent regression.

Fixes #29331

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

Before:

before--missing-hidden-fields.mov

After:

after--hidden-fields.mov

TESTING INSTRUCTIONS

  1. Create line chart with temporal x-axis column and COUNT(*) metric and a specific time grain like Month
  2. Switch the x-axis variable to categorical or numeric variable instead and click Update Chart
  3. Switch the x-axis variable back to the original variable.

ADDITIONAL INFORMATION

  • Has associated issue:
  • Required feature flags:
  • Changes UI
  • Includes DB Migration (follow approval process in SIP-59)
    • 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

@dosubot dosubot Bot added explore Namespace | Anything related to Explore requires:tests This PR requires tests labels Jun 24, 2024
@rusackas
Copy link
Copy Markdown
Member

Thank you!!!

@rusackas rusackas added the review:checkpoint Last PR reviewed during the daily review standup label Jun 24, 2024
@justinpark justinpark force-pushed the hotfix--hidden-control-value-discarded branch from a9ad6f2 to b705009 Compare June 24, 2024 21:10
@justinpark justinpark merged commit 160cece into apache:master Jun 24, 2024
@michael-s-molina michael-s-molina added the v4.0 Label added by the release manager to track PRs to be included in the 4.0 branch label Jun 25, 2024
@john-bodley john-bodley removed the review:checkpoint Last PR reviewed during the daily review standup label Jun 25, 2024
michael-s-molina pushed a commit that referenced this pull request Jun 26, 2024
@mistercrunch mistercrunch added 🍒 4.0.2 Cherry-picked to 4.0.2 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels labels Jul 24, 2024
@mistercrunch mistercrunch added the 🚢 4.1.0 First shipped in 4.1.0 label Nov 27, 2024
ratuldawar11 pushed a commit to grofers/superset that referenced this pull request Apr 14, 2026
* fix(permalink): adding anchor to dashboard permalink generation (apache#28744)

* fix: filters not updating with force update when caching is enabled (apache#29291)

(cherry picked from commit 527f1d2)

* fix(sqllab): invalid empty state on switch tab (apache#29278)

* fix(metastore-cache): prune before add (apache#29301)

(cherry picked from commit 172ddb4)

* fix: Remove recursive repr call (apache#29314)

(cherry picked from commit 9444c6b)

* fix: Cannot delete empty column inside a tab using the dashboard editor (apache#29346)

(cherry picked from commit ee52277)

* fix(explore): restored hidden field values has discarded (apache#29349)

(cherry picked from commit 160cece)

* chore: Rename Totals to Summary in table chart (apache#29360)

* fix(revert 27883): Excess padding in horizontal Bar charts (apache#29345)

(cherry picked from commit 708afb7)

* fix(explore): don't respect y-axis formatting (apache#29367)

* fix: adds the ability to disallow SQL functions per engine (apache#28639)

* chore: Adds 4.0.2 RC2 data to CHANGELOG.md

* fixes

* frontend fixes

* fix: cache api

---------

Co-authored-by: Jack <41238731+fisjac@users.noreply.github.com>
Co-authored-by: ka-weihe <k@weihe.dk>
Co-authored-by: JUST.in DO IT <justin.park@airbnb.com>
Co-authored-by: Ville Brofeldt <33317356+villebro@users.noreply.github.com>
Co-authored-by: Jessie R <j@scjr.me>
Co-authored-by: Michael S. Molina <70410625+michael-s-molina@users.noreply.github.com>
Co-authored-by: Daniel Vaz Gaspar <danielvazgaspar@gmail.com>
Co-authored-by: Michael S. Molina <michael.s.molina@gmail.com>
qfcwell pushed a commit to qfcwell/superset that referenced this pull request May 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels explore Namespace | Anything related to Explore requires:tests This PR requires tests size/M v4.0 Label added by the release manager to track PRs to be included in the 4.0 branch 🍒 4.0.2 Cherry-picked to 4.0.2 🚢 4.1.0 First shipped in 4.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Changing to non-temporal variable, then back, causes time grain to no longer function in 4.0.2rc2

5 participants