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

Add reconciliation status to the Module details page #518

Open
petar-cvit opened this issue Aug 21, 2024 · 3 comments
Open

Add reconciliation status to the Module details page #518

petar-cvit opened this issue Aug 21, 2024 · 3 comments
Labels
good first issue Good for newcomers react typescript UI Update on the UI

Comments

@petar-cvit
Copy link
Collaborator

Is your feature request related to a problem? Please describe.
Each time a Module reconciles, the reconciliation loop can produce errors. However, some of those errors are not critical and are listed in the reconciliationStatus field of the module when the module is fetched from the API.

The scope of the issue is to show reconciliation errors on the Module details page if any errors are present in the fetched Module.

Describe the solution you'd like
You can add a warning icon next to the module name if there are any errors, and add a popover that would show all errors on the module

@petar-cvit petar-cvit added good first issue Good for newcomers UI Update on the UI typescript react labels Aug 21, 2024
@pravintargaryen
Copy link

Please assign this to me. Will submit PR alongside #499

@petar-cvit
Copy link
Collaborator Author

Hey @pravintargaryen, thanks for the comment, but it would be better to keep them in separate PRs so it's easier to track

@Sheikh-Abubaker
Copy link
Contributor

Hey @petar-cvit I'd like to try this out!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers react typescript UI Update on the UI
Projects
None yet
Development

No branches or pull requests

3 participants