Skip to content

Better RetryErrors#553

Merged
N2D4 merged 28 commits into
devfrom
better-retry-errors
Apr 4, 2025
Merged

Better RetryErrors#553
N2D4 merged 28 commits into
devfrom
better-retry-errors

Conversation

@N2D4
Copy link
Copy Markdown
Contributor

@N2D4 N2D4 commented Mar 17, 2025


Important

Improves error handling by introducing nicify() for better error string representation and updates RetryError to use it, along with adding comprehensive tests.

  • Behavior:
    • Simplifies errorToNiceString() in errors.tsx by using nicify() directly.
    • Updates RetryError in results.tsx to use nicify() for error messages.
  • Functions:
    • Adds nicify() function in strings.tsx for improved error string representation.
    • Modifies comparePassword() in hashes.tsx to return Promise<boolean>.
  • Tests:
    • Adds strings.nicify.test.ts for comprehensive testing of nicify().
    • Removes strings.test.ts as it is no longer needed.
  • Misc:
    • Refactors deindent() and deindentTemplate() in strings.tsx for better readability.
    • Updates replaceAll() in strings.tsx to throw error if searchValue is empty.

This description was created by Ellipsis for 28f8ff6. It will automatically update as commits are pushed.

@N2D4 N2D4 self-assigned this Mar 17, 2025
@vercel
Copy link
Copy Markdown

vercel Bot commented Mar 17, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
stack-backend ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 4, 2025 11:36am
stack-dashboard ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 4, 2025 11:36am
stack-demo ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 4, 2025 11:36am

Comment thread apps/dashboard/src/app/(main)/(protected)/projects/[projectId]/layout.tsx Outdated
N2D4 and others added 2 commits March 20, 2025 10:18
…/layout.tsx

Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
Comment thread packages/stack-shared/src/utils/strings.tsx
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