Skip to content

Conversation

@trebitowski
Copy link
Collaborator

@trebitowski trebitowski commented Jan 12, 2026

Changes

  • Removes SDKKeyError and checkResponseSuccess in place of their shared versions from client-utils
  • Handles SDKKeyError and FormConflictError from client-utils. Form refresh logic is shared between both featheryClient and offlineHandler.
  • Instead of refreshing the form in checkResponseSuccess directly, it's in the error handler on thrown FormConflictErrors.

Testing

Form updates cause 409 error and refresh alert
Using invalid sdk logs sdkkeyerror to console

Deployment plan

  • Release client-utils changes
  • Update PR with new client-utils version
  • Release @feathery/react

Checklist before requesting a review

  • Cleaned up debug prints, comments, and unused code
  • Tested end to end
  • Included screenshots or walkthrough video of change if impacts UX

Related pull requests

Link other PRs here that are related to this change

https://github.com/feathery-org/client-utils/pull/3/commits

@trebitowski trebitowski force-pushed the fix/handle-client-utils-error branch from ea52bfa to b41e9cc Compare January 13, 2026 01:21
@trebitowski trebitowski merged commit 6f69575 into master Jan 13, 2026
6 checks passed
@trebitowski trebitowski deleted the fix/handle-client-utils-error branch January 13, 2026 01:27
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.

4 participants