Skip to content

Conversation

@villebro
Copy link
Member

@villebro villebro commented Apr 30, 2021

SUMMARY

If parents in the filter scope is undefined, an error is raised, making it impossible to change filter scope.

BEFORE

scope-before

AFTER

scope-after

TEST PLAN

ADDITIONAL INFORMATION

  • Has associated issue: closes [native filter] can not set filter scope: JS errors #14422
  • 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

Copy link
Member

@kgabryje kgabryje left a comment

Choose a reason for hiding this comment

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

LGTM

@codecov
Copy link

codecov bot commented Apr 30, 2021

Codecov Report

Merging #14426 (f57bce0) into master (1c16261) will increase coverage by 0.12%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #14426      +/-   ##
==========================================
+ Coverage   76.81%   76.93%   +0.12%     
==========================================
  Files         955      955              
  Lines       48249    48019     -230     
  Branches     6032     6033       +1     
==========================================
- Hits        37063    36945     -118     
+ Misses      10990    10876     -114     
- Partials      196      198       +2     
Flag Coverage Δ
javascript 71.57% <100.00%> (+0.20%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...ConfigModal/FiltersConfigForm/FilterScope/utils.ts 98.43% <100.00%> (ø)
superset-frontend/src/components/Icons/Icon.tsx 96.00% <0.00%> (-4.00%) ⬇️
superset-frontend/src/components/Alert/index.tsx 100.00% <0.00%> (ø)
...set-frontend/src/components/Icons/AntdEnhanced.tsx 87.50% <0.00%> (ø)
...nd/src/dashboard/components/FiltersBadge/index.tsx 100.00% <0.00%> (ø)
...re/components/controls/DatasourceControl/index.jsx 86.84% <0.00%> (ø)
superset-frontend/src/components/Icons/index.ts
superset-frontend/src/common/components/Card.tsx
superset-frontend/src/components/Card/index.tsx 100.00% <0.00%> (ø)
superset-frontend/src/components/Icons/index.tsx 100.00% <0.00%> (ø)
... and 2 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1c16261...f57bce0. Read the comment docs.

@villebro villebro merged commit 319125e into apache:master Apr 30, 2021
@villebro villebro deleted the villebro/native-scope branch April 30, 2021 09:20
@junlincc junlincc added the dashboard:native-filters Related to the native filters of the Dashboard label Apr 30, 2021
amitmiran137 pushed a commit that referenced this pull request May 2, 2021
amitmiran137 pushed a commit to nielsen-oss/superset that referenced this pull request May 3, 2021
* master: (38 commits)
  refactor(native-filters): allow cascading only for filter_select (apache#14441)
  test(maximize-chart): Add tests to maximize chart action (apache#14371)
  fix: fixing mysql error message (apache#14416)
  feat: Logic added to limiting factor column in Query model (apache#13521)
  change relationship (apache#14435)
  fix: bootstrap data permissions (apache#14348)
  fix: parse simple string error message values (apache#14360)
  chore: add stack trace to all calls of logger.error (apache#14382)
  update README with new docs and recordings (apache#14432)
  Renamed impyla from implya in impala.mdx and Renamed PIP package impyla from impala in index.mdx (apache#14425)
  fix(native-filters): fix filter scope error (apache#14426)
  feat: Adding limiting_factor column to Query model (apache#14234)
  feat: Add etag caching to dashboard APIs (apache#14357)
  chore: Moves Card to the components folder (apache#14139)
  feat: Dynamic imports for the Icons component (apache#14318)
  feat: Support env vars configuration for WebSocket server (apache#14398)
  fix: SQLLab role permissions (apache#14372)
  fix(native-filters): always show filters without dataset (apache#14409)
  fix error getting partitionQuery from table.partition (apache#14369)
  refactor: Boostrap to AntD - Tabs (apache#14048)
  ...
@rusackas rusackas added the bash! label May 12, 2021
QAlexBall pushed a commit to QAlexBall/superset that referenced this pull request Dec 29, 2021
@mistercrunch mistercrunch added 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 1.2.0 First shipped in 1.2.0 labels Mar 12, 2024
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 dashboard:native-filters Related to the native filters of the Dashboard preset:2021.17 preset-io size/XS v1.2 🚢 1.2.0 First shipped in 1.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[native filter] can not set filter scope: JS errors

5 participants