feat: Cross-referenced Dashboards in Chart list (Column + Filter)

[geido]

SUMMARY

This PR adds the ability to filter the Charts by the cross-referenced Dashboards. It also shows the referenced Dashboards in a new column.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

DEV.Superset.mp4

TESTING INSTRUCTIONS

1. Filter the Charts list by a Dashboard and make sure the list is filtered correctly
2. Add a Chart to multiple Dashboards, make sure the references are appearing in the "Dashboards added to" column
3. Add more Dashboards than what fits the "Dashboards added to" column. Make sure the tooltip is appearing
4. Add more than 20 Dashboards. Make sure the "Plus X more" is appearing in the tooltip

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

[codecov]

Codecov Report

Merging #21760 (ab591cf) into master (8f74e46) will decrease coverage by 0.00%.
The diff coverage is 50.00%.

❗ Current head ab591cf differs from pull request most recent head 7774d5c. Consider uploading reports for the commit 7774d5c to get more accurate results

@@            Coverage Diff             @@
##           master   #21760      +/-   ##
==========================================
- Coverage   66.86%   66.85%   -0.01%     
==========================================
  Files        1803     1805       +2     
  Lines       68996    69053      +57     
  Branches     7349     7368      +19     
==========================================
+ Hits        46132    46165      +33     
- Misses      20971    20991      +20     
- Partials     1893     1897       +4     

Flag	Coverage Δ
javascript	53.20% <56.33%> (+<0.01%)	⬆️

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

Impacted Files	Coverage Δ
...rd/components/nativeFilters/FilterCard/NameRow.tsx	85.71% <ø> (ø)
superset/views/core.py	76.06% <ø> (ø)
superset/migrations/shared/utils.py	36.06% <12.50%> (ø)
superset-frontend/src/hooks/useTruncation/index.ts	44.73% <16.66%> (ø)
...perset-frontend/src/views/CRUD/chart/ChartList.tsx	56.61% <42.10%> (-2.36%)	⬇️
.../explore/exploreUtils/getParsedExploreURLParams.ts	83.33% <50.00%> (-8.10%)	⬇️
...tend/src/components/ListView/CrossLinksTooltip.tsx	77.77% <77.77%> (ø)
...et-frontend/src/components/ListView/CrossLinks.tsx	88.23% <88.23%> (ø)
...nents/nativeFilters/FilterCard/DependenciesRow.tsx	80.00% <100.00%> (+1.05%)	⬆️
...d/components/nativeFilters/FilterCard/ScopeRow.tsx	61.11% <100.00%> (+2.28%)	⬆️
... and 7 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[kgabryje]

Left some minor comments. Tagging @kasiazjc for truncation behaviour review 🙂

[kgabryje]

/testenv up FEATURE_CROSS_REFERENCES=true

[github-actions]

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

[geido]

/testenv up FEATURE_CROSS_REFERENCES=true

[michael-s-molina]

Thanks for the PR @geido! I left some first-pass comments.

[github-actions]

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

[zhaoyongjie]

Tested in my local. work as expected.

[michael-s-molina]

Can you make the +n hoverable? It feels natural to hover the +n to see the hidden dashboards.

Screen.Recording.2022-10-12.at.7.48.04.AM.mov

[geido]

/testenv up FEATURE_CROSS_REFERENCES=true

[github-actions]

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

[jinghua-qa]

LGTM

[michael-s-molina]

LGTM. Thanks for addressing all the comments!

[github-actions]

Ephemeral environment shutdown and build artifacts deleted.

[john-bodley]

@geido I believe there's an issue with this feature when backend querying is required, i.e., the following request,

 http://localhost:5000/api/v1/chart/?q=(columns:!(dashboard_title,id),filters:!((col:dashboards,opr:rel_m_m,value:d)),keys:!(none),order_column:dashboard_title,order_direction:asc,page:0,page_size:100)

throws the following FAB error:

"Invalid order by column: dashboard_title"

[geido]

@geido I believe there's an issue with this feature when backend querying is required, i.e., the following request,

 http://localhost:5000/api/v1/chart/?q=(columns:!(dashboard_title,id),filters:!((col:dashboards,opr:rel_m_m,value:d)),keys:!(none),order_column:dashboard_title,order_direction:asc,page:0,page_size:100)

throws the following FAB error:

"Invalid order by column: dashboard_title"

Thanks @john-bodley I will have a look