bvaughn / bvaughn/forms-js

Integrate apiCheck for form-validation and form-layout schema validation

Open
#3 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
JavaScript
Stars
11
Forks
1
PR merge metrics
No merged PRs in 30d

Description

@kentcdodds 's [apiCheck.js](https://github.com/kentcdodds/apiCheck.js) seems promising for runtime validation of both form-validation and form-layout schemas. We should integrate with it for the debug build of Forms JS and remove it entirely for the production build*.

\* This is assuming we don't want production validation due to a negative impact on performance. As we have not verified how much of a negative impact apiCheck.js will have on performance, it is fair to question this decision.

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reviewing apiCheck.js and the existing form-validation and form-layout schema definitions. Then inspect the debug and production build configuration to determine where validation should be included or removed. Done means both schemas are validated in debug builds, omitted from production builds, and the performance trade-off is documented or resolved.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
frontend
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.