8000 Add schema adapter system by danielweinmann · Pull Request #373 · seasonedcc/remix-forms · GitHub
[go: up one dir, main page]
Skip to content

Add schema adapter system #373

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
Closed
wants to merge 7 commits into from
Closed

Add schema adapter system #373

wants to merge 7 commits into from
+414 −234

Conversation

danielweinmann
Copy link
Contributor

Summary

  • add adapter interface for schema libraries
  • implement zod3Adapter using zod
  • expose adapter from entrypoint
  • integrate adapter with SchemaForm
  • update public API tests

Testing

  • npm run lint-fix
  • npm run lint
  • npm run tsc
  • npm run test
< 8000 input type="hidden" name="authenticity_token" value="uYK4_BlOvtp_KZPLOrBCMwJ034_2OFOIYZN7WDERFSPGDeOH4QmaYIyIXT-42G0Xfrf9Dzx3ViT3TKzUmyv3Iw" autocomplete="off" />

@danielweinmann danielweinmann marked this pull request as draft May 28, 2025 18:26
@danielweinmann danielweinmann force-pushed the codex/create-schemaadapter-with-zod3-implementation branch from ea2166e to f5cc95a Compare May 28, 2025 19:38
@danielweinmann
Refactor type names and remove zod imports
09e1f85
@danielweinmann
Merge pull request #376 from seasonedcc/codex/refactor-to-remove-zod-…
9f5a19e 
…dependency

Refactor schema internals away from zod specifics
@danielweinmann
Introduce generic schema types
950da92
@danielweinmann
Revert apps/web changes from last commit
4f54db1 
Keep the packages/remix-forms changes from the previous commit while
undoing the changes made to apps/web files.
@danielweinmann
Merge pull request #377 from seasonedcc/codex/refactor-types-to-decou…
5339caa 
…ple-from-zod

Refactor adapter types
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
codex
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@danielweinmann
0