Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found
Not Found

All comments

  • Farhan• 250

    @ashalfarhan

    Posted

    This is cool!

    Feedback:

    • Instead of using a div to a clickable element, use button in this line
    • Add a loading state on the button to prevent calling the API while another request is still ongoing
    1
  • Farhan• 250

    @ashalfarhan

    Posted

    Nice one! Few feedbacks:

    • Instead of adding an onClick handler to a div, use a button in this line .
    • Use the regular if statement in this line. This is just a preference.

    Marked as helpful

    1
  • Farhan• 250

    @ashalfarhan

    Posted

    I think, instead of importing directly in your html, import it in your cra entry file (index.js), so they will be minified with webpack.

    0