Skip to content
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

AIP-38 | List Import Errors #43711

Closed
bbovenzi opened this issue Nov 5, 2024 · 2 comments · Fixed by #44021
Closed

AIP-38 | List Import Errors #43711

bbovenzi opened this issue Nov 5, 2024 · 2 comments · Fixed by #44021
Assignees
Labels
AIP-38 Modern Web Application area:UI Related to UI/UX. For Frontend Developers.

Comments

@bbovenzi
Copy link
Contributor

bbovenzi commented Nov 5, 2024

Image

Use the auto-generated list import errors query in the new UI.

Add a button to the Dashboard page saying how many import errors there are (don't show the button if there are 0).
Clicking the button should open a large modal listing all import errors. For a long list, the user should be able to scroll inside the modal.

@bbovenzi bbovenzi converted this from a draft issue Nov 5, 2024
@bbovenzi bbovenzi added area:UI Related to UI/UX. For Frontend Developers. AIP-38 Modern Web Application labels Nov 5, 2024
@shubhamraj-git
Copy link
Contributor

Hey could you please assign this to me.

  1. Dashboard contains only health for now, will add the stats.
  2. Inside the stats, we will have different buttons, one will be "Import errors"
  3. Existing API : https://localhost:29091/docs#/Import%20Error/get_import_errors
  4. We will show list as DAG display name and accordion to view the import errors.
  5. I was thinking about modal size to 5, scrollable to 10 and pagination effect.
  6. Future work: Will support any search / sort filter on the modal.

@bbovenzi
Copy link
Contributor Author

Assigned and those 6 steps make sense to me!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AIP-38 Modern Web Application area:UI Related to UI/UX. For Frontend Developers.
Projects
Development

Successfully merging a pull request may close this issue.

2 participants