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

Add 'Cancel' to login form #162

Closed
patmbolger opened this issue Jan 12, 2016 · 3 comments
Closed

Add 'Cancel' to login form #162

patmbolger opened this issue Jan 12, 2016 · 3 comments
Assignees
Labels
Milestone

Comments

@patmbolger
Copy link
Contributor

As a user
I want to be able to cancel out of the login form
And be back on the page from which I clicked "Login"

@amba178 amba178 self-assigned this Jan 14, 2016
@patmbolger
Copy link
Contributor Author

Enhance the user login feature:

  • Allow user to cancel login and return to page the user was on when he/she clicked the 'login' button
  • After successful login, the user should return to page the user was on when he/she clicked the 'login' button

@amba178
Copy link

amba178 commented Jan 15, 2016

The second part a bit complex, it forces us maybe to override session control for devise, and utilize session. @patmbolger : Do you have any rough idea how to go about it.

@patmbolger
Copy link
Contributor Author

Suggest these as possible approaches:

  1. Possibly redirect back to prior page - here are some ideas: http://stackoverflow.com/questions/2139996/how-to-redirect-to-previous-page-in-ruby-on-rails

  2. Change login form to be a pop-up (modal) dialog box over the current page. Look at what I did for the company registration 'deny' action. (register a company registration request as a site visitor, login as agency admin, click on company name in the admin home page, then click 'Deny' button).

@amba178 amba178 added the ready label Jan 15, 2016
@joaopapereira joaopapereira added this to the Sprint 8 milestone Jan 15, 2016
@amba178 amba178 added in progress and removed ready labels Jan 21, 2016
joaopapereira added a commit to AgileVentures/MetPlus_PETS that referenced this issue Jan 28, 2016
@amba178 amba178 closed this as completed Jan 28, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants