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

Thyme UI integration #20

Open
wants to merge 14 commits into
base: master
Choose a base branch
from
Open

Thyme UI integration #20

wants to merge 14 commits into from

Conversation

shubhangirai8
Copy link
Contributor

Implemented ThymeLeaf for Login and Registration

@shubhangirai8
Copy link
Contributor Author

Corrections Needed-

  1. The code is not working as expected It should show login and signup both to a non-loggedin user. Currently it shows either Sign in OR Sign up. (For a non-logged in user)
  2. The sec:authorize tag are applied only to the the anchor tags . This is not the case tough, rather the Widget of human is showed only for a loggedin user. Here the sec:authorize constraint is not applied to the widget as a whole
  3. The location of Sign Up is shifted up due to sec:authorize applied only to the anchor tags.

Please check and run the code locally and ensure its working perfectly before pull request.

@Shubham-Shingate database was not up yesterday, so wasn't able to test. Will test in a while

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants