Double click on "Yes it's safe" breaks whole app - Closes #642

[yasharAyari]

What was the problem?

As title suggests if we double clicked Yes it's safe button in Registration process, the component throwed error.

How did I fix it?

Disable the button after first click.

How to test it?

Review checklist

• The PR solves Double click on "Yes it's safe" breaks whole app  #642
• All new code is covered with unit tests
• All new features are covered with e2e tests
• All new code follows best practices

[reyraa]

Thank you Yashar

[alepop]

@yasharAyari hello! I think, there are a lot of place in the app, then you need to prevent double click, maybe better is to make a HOC component for click elements, that will allow only one click.

[reyraa]

@alepop Correct. thanks for your suggestion. we can create an HOC for all buttons presenting double click event. I'll create a ticket to fix it with next release.

[yasharAyari]

@alepop I don't agree with you . I think we can handle all of them with 'disabled' property in button component like this one.

[alepop]

@yasharAyari you can create a HOC that will set the disabled attribute automatically on the first click. In my opinion, it is better than add all this code in each component.