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

Am/feature inactivity timeout #1572

Merged
merged 6 commits into from
Apr 26, 2021
Merged

Conversation

azharem
Copy link
Contributor

@azharem azharem commented Apr 23, 2021

  1. Better UX experience for when session ends due to inactivity
I want GovReady-Q to automatically navigate me to the login page instead of continuing to display in the browser window the previously loaded GovReady-Q page.

Timeout should probably be a system settings in the database.

I know this is done when:

The session times out due to inactivity
Activity is identified by no up or down movement or clicks within the page 
The time amount for timeout has to be an adjustable variable 
CMS settings 30 mins 
Once logged back in user redirects to  Home page 
  1. Time Alert - Better UX experience for when session ends due to inactivity
As a GovReady user, when my session times out due to inactivity,
I want to receive an alert that my session was going to time out in 5 minutes.

So that I wont be logged out while still working on stuff. 

I know this is done when: 

The system will send a notification when there is a certain number of minutes of inactivity.  This number of minutes is configurable in environment deployment setting and by default is set to 20.

@azharem azharem requested a review from davidpofo April 23, 2021 14:53
install.py Show resolved Hide resolved
requirements.txt Outdated Show resolved Hide resolved
siteapp/settings.py Outdated Show resolved Hide resolved
siteapp/urls.py Show resolved Hide resolved
siteapp/tests.py Outdated Show resolved Hide resolved
siteapp/settings.py Outdated Show resolved Hide resolved
@davidpofo
Copy link
Contributor

@azharem Make sure to address the branch conflicts and you are go to go!

Copy link
Contributor

@davidpofo davidpofo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm after I resolved the conflicts with develop

@davidpofo davidpofo merged commit 8299b1e into develop Apr 26, 2021
@davidpofo davidpofo deleted the am/feature_inactivity_timeout branch April 26, 2021 21:25
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.

3 participants