Multi-step form: React | TypeScript | SCSS

Community feedback

• 

  Mukwende•1,370

  @mukwende2000

  Posted almost 2 years ago

  Hello, this is my honest feedback, first react router was not needed for this project as the different part of the form are not routes, by splitting the form part into routes, you have just made each part a different no wonder you have form on every route. This is one form and when submitted should submit all the data as one group of data from one form, but as it is, the way you have structured this mean each part is its own form.

  41

  Marked as helpful
• 

  Aryan Kumar•390

  @aryanda1

  Posted almost 2 years ago

  I noticed two problems while testing your website.

  1. when i enter some valid in name and email, and a invalid number, and click on next step, the website refresh and erases all inputs.
  2. if i selected some addon, and after changing plan, the addon gets removed.

  12
• 

  Mukwende•1,370

  @mukwende2000

  Posted almost 2 years ago

  Having something like this

  <form>
  some inputs
  </form>
  
  <form>
  some other inputs
  </form>
  

  Makes this two forms regardless of any state, and each form has it own onSubmit event, forms have everything to do with html and nothing to do with react, so you should never compromise html structure because of any framework. As for routing, it is for navigating between pages and not for dividing forms. That was what i meant. Happy Coding.... Have an amazing day.

  01