feat: alerts and reports modal with dashboard filters and tabs [WIP]

[lilykuang]

SUMMARY

• this pr is POC for the SIP of alerts and reports modal with dashboard filters and tabs and not going to be merged

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

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

[john-bodley]

@lilykuang I realize that this is in draft mode, but my understanding was that the Alerting + Reporting redesign was going to be tied to a SIP. Is this merely a PoC for use in the SIP?

[codecov]

Codecov Report

Attention: 54 lines in your changes are missing coverage. Please review.

Comparison is base (1fbc762) 69.18% compared to head (2497ac5) 69.13%.

Files	Patch %	Lines
...ures/alerts/components/DashboardFiltersAndTabs.tsx	2.12%	46 Missing ⚠️
...-frontend/src/features/alerts/AlertReportModal.tsx	28.57%	4 Missing and 1 partial ⚠️
...et-frontend/src/dashboard/actions/nativeFilters.ts	50.00%	1 Missing ⚠️
superset-frontend/src/dataMask/actions.ts	66.66%	1 Missing ⚠️
superset-frontend/src/dataMask/reducer.ts	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master   #26141      +/-   ##
==========================================
- Coverage   69.18%   69.13%   -0.05%     
==========================================
  Files        1945     1946       +1     
  Lines       75948    76008      +60     
  Branches     8458     8476      +18     
==========================================
+ Hits        52544    52550       +6     
- Misses      21219    21272      +53     
- Partials     2185     2186       +1     

Flag	Coverage Δ
javascript	56.44% <10.00%> (-0.08%)	⬇️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[lilykuang]

yes, @john-bodley this pr is a PoC for Alerts and Report modal and it's not going to be merged

[eschutho]

/testenv up

[github-actions]

@eschutho Ephemeral environment spinning up at http://34.215.251.39:8080. Credentials are admin/admin. Please allow several minutes for bootstrapping and startup.

[eschutho]

/testenv up FEATURE_ALERT_REPORTS=true

[github-actions]

@eschutho Ephemeral environment spinning up at http://54.187.177.192:8080. Credentials are admin/admin. Please allow several minutes for bootstrapping and startup.

[lilykuang]

/testenv up FEATURE_ALERT_REPORTS=true

[github-actions]

@lilykuang Ephemeral environment spinning up at http://34.209.239.80:8080. Credentials are admin/admin. Please allow several minutes for bootstrapping and startup.

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 10.00000% with 54 lines in your changes missing coverage. Please review.

Project coverage is 69.13%. Comparing base (1fbc762) to head (2497ac5).
Report is 1662 commits behind head on master.

Files with missing lines	Patch %	Lines
...ures/alerts/components/DashboardFiltersAndTabs.tsx	2.12%	46 Missing ⚠️
...-frontend/src/features/alerts/AlertReportModal.tsx	28.57%	4 Missing and 1 partial ⚠️
...et-frontend/src/dashboard/actions/nativeFilters.ts	50.00%	1 Missing ⚠️
superset-frontend/src/dataMask/actions.ts	66.66%	1 Missing ⚠️
superset-frontend/src/dataMask/reducer.ts	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master   #26141      +/-   ##
==========================================
- Coverage   69.18%   69.13%   -0.05%     
==========================================
  Files        1945     1946       +1     
  Lines       75948    76008      +60     
  Branches     8458     8476      +18     
==========================================
+ Hits        52544    52550       +6     
- Misses      21219    21272      +53     
- Partials     2185     2186       +1     

Flag	Coverage Δ
javascript	56.44% <10.00%> (-0.08%)	⬇️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.