Skip to content

Filter duplicate error messages#3927

Merged
joshri merged 1 commit intomainfrom
filter-errors
Aug 9, 2023
Merged

Filter duplicate error messages#3927
joshri merged 1 commit intomainfrom
filter-errors

Conversation

@joshri
Copy link
Copy Markdown
Contributor

@joshri joshri commented Aug 7, 2023

Closes #3139

Quick fix for OSS errors displaying the same error like seven or eight times. It would be useful for OSS to adopt Enterprise's cleaner error handling in the future

@joshri joshri added the area/ui Issues that require front-end work label Aug 7, 2023
@joshri joshri requested a review from opudrovs August 7, 2023 18:46
@makkes
Copy link
Copy Markdown
Member

makkes commented Aug 8, 2023

@joshri is this related to #3634 as well?

@joshri
Copy link
Copy Markdown
Contributor Author

joshri commented Aug 8, 2023

@makkes not quite, this only removes errors that have the exact same message from the list. Jordan is going to take on the permission errors situation

@makkes
Copy link
Copy Markdown
Member

makkes commented Aug 8, 2023

Thanks. Nice one in any case!!

@joshri joshri merged commit c147a72 into main Aug 9, 2023
@joshri joshri deleted the filter-errors branch August 9, 2023 22:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/ui Issues that require front-end work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants