Skip to content

Conversation

@tamaratumeh
Copy link

No description provided.

Copy link
Contributor

@dvmhmdsd dvmhmdsd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your awesome job, please resolve the below comment to merge this PR

function simulateInputChange(callback) {
callback('updated');
}
simulateInputChange(formHandler.onChange.bind(formHandler));
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

could u please use arrow function as another solution here? and explain why it could work

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I fixed it

// another solution
const formHandler = {
value: 'initial',
onChange: (newValue) => {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

u should not edit the original code

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants