<Form/> cannot submitted via ‘action’ property
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 35/100
- Tipo di issue
- Bug
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Ferma
- Stack tecnologico
- react, typescript
- Ambito
- frontend
Direzione di ricerca
Inizia in src/Form.tsx, nella gestione dell’evento submit intorno alle righe 164–165, quindi esegui il repository dimostrativo collegato per riprodurre l’invio nativo non riuscito. La correzione dovrebbe consentire a un modulo con action e method di essere inviato tramite il meccanismo nativo, preservando il comportamento di validazione e la gestione di onFinish.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
I'm currently experimenting with a login form implemented using rc-field-form at its core that interacts with an OAuth2/OIDC implementation that expects a traditional server side rendered app for the login form. For this reason, I want to not attach an onFinish() callback to the form, but instead submit it via the native <form> submit mechanism, by setting the action property on the form, like so:
<Form name="test-form" action="https://httpstat.us/200" method="post">
<Field name="test-field">
<input placeholder="test field" value={""}/>
</Field>
<button type="submit" value={"submit"}>
Submit
</button>
</Form>
Since FormProps transitively extends React.FormHTMLAttributes I was expecting this to work, but of course it doesn't because here the event propagation of the submit click is stopped in favour of delegating processing to the form instance. I think rc-field-form's behaviour would be less surprising if event propagation were only stopped if:
- Form validation fails
- An on finish callback is actually attached to the form.
Of course, the alternative would be to omit action (and probably method) from FormProps, though that hardly seems satisfactory to me.
A demonstrator reproducing the bug can be found here: https://github.com/ngrewe/rc-field-form-bug
Thanks,
Niels
- Lingua principale
- TypeScript
- Stelle
- 1k
- Fork
- 286
- Merge medio
- 4g 16h
- PR unite (30g)
- 1
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di react-component/field-form
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 25/100
react-component/field-form#768 · 5 commenti ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 25/100
react-component/field-form#751 ·
-
Add getFieldsWarning API Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 42/100
react-component/field-form#749 ·
-
useWatch支持dynamic path Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
react-component/field-form#746 ·
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
react-component/field-form#733 · 1 commento ·
Tutte le issue di react-component/field-form
Issue simili
-
bug(cli): hapi doctor inline-media prints a fabricated B:\ helper-script path in packaged installs Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
-
Crush Aperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 85/100
catppuccin/catppuccin#3125 ·
-
Add a SECURITY.md Aperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 90/100
ElementsProject/cln-application#167 · 1 commento · 1 reazione ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100
Quantco/pnpm-licenses#17 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100