-
Notifications
You must be signed in to change notification settings - Fork 14.1k
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: Direct Linking issue on report list: 404 status code. #20033
fix: Direct Linking issue on report list: 404 status code. #20033
Conversation
Codecov Report
@@ Coverage Diff @@
## master #20033 +/- ##
==========================================
- Coverage 66.33% 66.30% -0.04%
==========================================
Files 1713 1713
Lines 64083 64084 +1
Branches 6734 6734
==========================================
- Hits 42509 42490 -19
- Misses 19863 19883 +20
Partials 1711 1711
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
(cherry picked from commit 9cf9f97)
🏷️ preset:2022.19 |
SUMMARY
The current superset throws 404 when you try to go to report page directly (
/report/list
)The problem was that the ReportView class was not registered into FAB.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
BEFORE:
[DEV] Superset - 11 May 2022 - Watch Video
AFTER:
[DEV] Superset - 11 May 2022 - Watch Video
TESTING INSTRUCTIONS
Another method:
ADDITIONAL INFORMATION