Skip to content
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.

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

Integrate the captcha component with the reactive forms #17

Open
AlexITC opened this issue Sep 1, 2018 · 0 comments
Open

Integrate the captcha component with the reactive forms #17

AlexITC opened this issue Sep 1, 2018 · 0 comments
Labels
alerts-ui Changes required on the alerts-ui project good first issue help wanted

Comments

@AlexITC
Copy link
Owner

AlexITC commented Sep 1, 2018

The captcha component is not integrated with the reactive forms.

Expected behavior

We should be able to keep disabled the submit button on reactive forms is the captcha hasn't been solved.

Actual behavior

The captcha component is not integrated with reactive forms, which implies that it is not considered as a form element, leading to the submit button being clickable even is the captcha hasn't been solved.

Example:
login

As concisely as possible, describe the observed behavior.

Steps to reproduce the behavior

  1. Go to the login page (https://cryptocoinalerts.net/login).
  2. Type a valid email and password.
  3. The login button is enabled while it shouldn't because the captcha isn't solved.Please list all relevant steps to reproduce the observed behavior.
@AlexITC AlexITC added enhancement help wanted good first issue alerts-ui Changes required on the alerts-ui project labels Sep 1, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
alerts-ui Changes required on the alerts-ui project good first issue help wanted
Projects
None yet
Development

No branches or pull requests

1 participant