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.

Sign up for GitHub

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

Jump to bottom

refactor: collection ui now displays 3 errors (count of additional) and link to file path containing the full error list #189

Draft
wants to merge 2 commits into
base: develop
Choose a base branch
from
Draft

refactor: collection ui now displays 3 errors (count of additional) and link to file path containing the full error list #189

wants to merge 2 commits into from
+61 −3

Conversation

graza-io
Copy link
Contributor

@graza-io graza-io commented Feb 3, 2025

Closes #180

@graza-io
refactor: collection ui now displays 3 errors (count of additional) a…
268af68 
…nd link to file path containing the full error list
@graza-io graza-io requested a review from kaidaguerre February 3, 2025 14:24
@graza-io
Copy link
Contributor Author

graza-io commented Feb 4, 2025 •
edited
Loading

image

Our error messages may need a little bit of review - although error during enrichment is some dummy text I've used when forcing errors for the example.

@graza-io graza-io marked this pull request as draft February 4, 2025 16:18
@graza-io
Copy link
Contributor Author

graza-io commented Feb 4, 2025

Converted to Draft as we're going to review error messages prior to merging this as currently they aren't the most helpful things to display.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kaidaguerre kaidaguerre Awaiting requested review from kaidaguerre

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
1 participant
@graza-io