New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Rename `QuantityTextInput` event handlers to use `on*` format #414

Open

jeremyckahn opened this issue May 22, 2023 · 0 comments

Owner

jeremyckahn commented May 22, 2023

QuantityInput event handlers are using the wrong format, per https://jaketrent.com/post/naming-event-handlers-react.

farmhand/src/components/QuantityInput/QuantityInput.js

Lines 29 to 35 in 6d2a795

    
           // TODO: Rename event handlers to use on* format 
        
           const QuantityTextInput = ({ 
        
             handleSubmit, 
        
             handleUpdateNumber, 
        
             maxQuantity, 
        
             value, 
        
           }) => (

The text was updated successfully, but these errors were encountered:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment