rusackas commented on PR #30016: URL: https://github.com/apache/superset/pull/30016#issuecomment-2335037978
As discussed with @michael-s-molina, I made one of several steps so that we might get this PR merged, and follow up with more. The results of the displayed at-risk items are now limited to 10. This gives more context than we had before, without causing massive scrolling problems. <img width="614" alt="image" src="https://github.com/user-attachments/assets/52c4828a-8574-4188-96be-4facb89aa548";> Next steps may include: • Displaying/counting more at-risk assets (saved queries, alerts/reports, etc) • Making the number if displayed items configurable • Adding "view more" link to load/paginate more items • Adding search/filters for the list • Adjusting the API to _also_ limit to 10 items (or as configured) and thus reduce a pre-existing performance bottleneck (CC @mistercrunch who offered to chip in here) • Add a spinner in case there _is_ a bottleneck. ... but for now, users merely have a bit more context on what's at risk, which was my original (simplistic) hope, even if the rabbit hole goes _a lot_ deeper from here. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
