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

login view update #178

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

login view update #178

wants to merge 6 commits into from

Conversation

sreetu
Copy link

@sreetu sreetu commented Apr 24, 2021

Related Issuse

  • Info about Issue or bug
  • To add mandatory field options in login inputs

Closes: #166 [issue number that will be closed through this PR]

Describe the changes you've made

A clear and concise description of what you have done to successfully close your assigned issue. Any new files? or anything you feel to let us know!
I have added 'requied' tag in the input forms of the login page to make them as madatory field

Checklist:

  • My code follows the style guidelines of this project.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • My changes generate no new warnings.

Screenshots

Original Updated
original screenshot updated screenshot

@sreetu
Copy link
Author

sreetu commented Apr 24, 2021

I have resolved the branch conflicts which were arising previously. Kindly review now sir @khannakshat7

@khannakshat7
Copy link
Owner

@sreetu Please resolve the conflicts

@sreetu
Copy link
Author

sreetu commented May 18, 2021

I have deleted the duplicate code and so, conflict has been resolved.

@khannakshat7
Copy link
Owner

@sreetu Any update on this?

@sreetu
Copy link
Author

sreetu commented May 22, 2021

@sreetu Any update on this?

@khannakshat7 sir, I have resolved the conflicts.

@arpit456jain
Copy link
Contributor

I think there is some mistake in this PR have a look
actual

after this

@arpit456jain
Copy link
Contributor

this PR is removing one of the most important code on that page
@khannakshat7 please look into this

@arpit456jain
Copy link
Contributor

there is a code for error / success msg after the div having id myTabContent but this PR is removing it.

Please allow me to make a PR for this

@arpit456jain
Copy link
Contributor

@khannakshat7 please assign me

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.

To add mandatory field option in login page
3 participants