Skip to content

fix(contrib): improve contrib backend error messages#2092

Open
n1k0 wants to merge 4 commits intomasterfrom
fix/catch-log-gh-api-contrib-errors
Open

fix(contrib): improve contrib backend error messages#2092
n1k0 wants to merge 4 commits intomasterfrom
fix/catch-log-gh-api-contrib-errors

Conversation

@n1k0
Copy link
Copy Markdown
Member

@n1k0 n1k0 commented Apr 15, 2026

This patch fixes a poorly worded error message rendered when the backend fails (here, the food2 scope doesn't have any examples json file just yet). Instead of a 400 Bad Request cryptic message rendered to end users, we now have something slightly more informative:

image

@n1k0 n1k0 requested a review from vjousse April 15, 2026 21:43
@n1k0 n1k0 changed the title fix(contrib): catch review assignment error, gh api=hell fix(contrib): catch review assignment error Apr 16, 2026
@n1k0 n1k0 removed the request for review from vjousse April 16, 2026 08:48
@n1k0
Copy link
Copy Markdown
Member Author

n1k0 commented Apr 16, 2026

There are still issues I'll keep working on, hold on reviewing, Edit: solved, reassigning for review

@n1k0 n1k0 requested a review from vjousse April 16, 2026 09:52
@n1k0 n1k0 force-pushed the fix/catch-log-gh-api-contrib-errors branch from 9124eae to 32337c9 Compare April 16, 2026 09:53
@n1k0
Copy link
Copy Markdown
Member Author

n1k0 commented Apr 16, 2026

Well, while I'm at it, I'm gonna add an examples.json file for food2. Shouldn't be much more work.

Edit: done in a2d9357

@n1k0 n1k0 force-pushed the fix/catch-log-gh-api-contrib-errors branch from a2d9357 to 9cdcd70 Compare April 16, 2026 10:17
@n1k0 n1k0 force-pushed the fix/catch-log-gh-api-contrib-errors branch from 9cdcd70 to d19eff5 Compare April 16, 2026 10:21
@n1k0 n1k0 changed the title fix(contrib): catch review assignment error fix(contrib): improve contrib backend error messages Apr 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant