-
Notifications
You must be signed in to change notification settings - Fork 0
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
Feat/webapp/new login #64
Conversation
…n the login.jsx the app is now responsive and follows the wireframe.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All seems to be correct.
I'm currently waiting for the bug in #69 to be solved, as I cannot work towards fixing the test that fails without it. I'll approve when it is. |
…w and hide button. In buttonEf changing the parameters received.
Quality Gate passedIssues Measures |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The requested changes seem to have been included. Approved.
Creating the login view with the corresponding tests and internationalisation. Two of the tests cannot pass.