Interactive card component with React and SASS

Community feedback

• 

  S MD suleman•3,530

  @sulemaan7070Posted almost 2 years ago

  hey Sarosh Farhan😄, great job on completing the challenge... design vise you did a remarkable job I would recommend you to limit the number of characters to be 2 in the case of YY and MM and 3 characters in the case of CVC... everything else seems good. Great job! happy coding💯🔥

  0

  ---

  

  Sarosh Farhan•270

  @saroshfarhanPosted almost 2 years ago

  @sulemaan7070 Thank you so much for the feedback. I have added a limit of 2 characters and 3 characters for YY and MM and CVC seems like I have not added an error condition to show the message. But it is not moving to the completed state with 3 numbers in YY and MM and more than 3 in CVC. So that case is handled, it seems like I need to add the error message. Thank you so much for the feedback and for taking the time to test my app!!

  1

  

  Sarosh Farhan•270

  @saroshfarhanPosted almost 2 years ago

  @sulemaan7070 I have now added the checks and proper validation. Thanks for the feedback again!!

  1