Uploaded image for project: 'Sakai'
  1. Sakai
  2. SAK-8152

Sakai should warn a user before the user's session times out and data is inadvertently lost

    Details

    • Type: Task Task
    • Status: Closed
    • Priority: Major Major
    • Resolution: Incorporated
    • Affects Version/s: 2.2.0, 2.2.1, 2.2.2, 2.2.3, 2.3.0, 2.3.1, 2.3.2, 2.4.0, 2.4.1, 2.5.0
    • Fix Version/s: None
    • Component/s: Global
    • Labels:
      None

      Description

      Goals:
      User can indicate he/she is still using Sakai before session times out

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            Wytze Koopal added a comment -

            Seems not like a new tool to me. Might be a user preference, that could be defaulted perhaps by admin. And the User can override this by setting his or her preference for session time out if needed.

            Show
            Wytze Koopal added a comment - Seems not like a new tool to me. Might be a user preference, that could be defaulted perhaps by admin. And the User can override this by setting his or her preference for session time out if needed.
            Hide
            Lance Speelmon added a comment -

            Ian, we have tested this code locally and it is working well. We are ready for a branch merge back into trunk when you have reviewed the code. Thanks! L

            Show
            Lance Speelmon added a comment - Ian, we have tested this code locally and it is working well. We are ready for a branch merge back into trunk when you have reviewed the code. Thanks! L
            Hide
            Ian Boston added a comment -

            Could you merge the current trunk into your branch and test

            in your branch working directory do
            svn merge -r39089:HEAD https://source.sakaiproject.org/svn/portal/trunk .

            There are some changes in trunk that might hit you. The code has moved from the $( notation to jQuery(

            eg

            • $('.more-tab').addClass('more-active');
              + jQuery('.more-tab').addClass('more-active');

            I think you have $( in your branch.

            Once that is done, it you should take it to someone from Fluid just to check the UI.

            Other than that, Im cool to merge.

            Ian

            Show
            Ian Boston added a comment - Could you merge the current trunk into your branch and test in your branch working directory do svn merge -r39089:HEAD https://source.sakaiproject.org/svn/portal/trunk . There are some changes in trunk that might hit you. The code has moved from the $( notation to jQuery( eg $('.more-tab').addClass('more-active'); + jQuery('.more-tab').addClass('more-active'); I think you have $( in your branch. Once that is done, it you should take it to someone from Fluid just to check the UI. Other than that, Im cool to merge. Ian
            Hide
            Ian Boston added a comment -

            see comments above.

            Show
            Ian Boston added a comment - see comments above.
            Hide
            Lance Speelmon added a comment -

            Joe - let me know if you need any assistance in Ian's first request. Thanks, L

            Show
            Lance Speelmon added a comment - Joe - let me know if you need any assistance in Ian's first request. Thanks, L
            Hide
            Lance Speelmon added a comment -

            Colin,

            Ian suggested that Fluid take a look at this new capability we have developed. We would like to get this merged back into trunk ASAP. We have this running on a test server if you would like to kick the tires. Thanks! L

            Show
            Lance Speelmon added a comment - Colin, Ian suggested that Fluid take a look at this new capability we have developed. We would like to get this merged back into trunk ASAP. We have this running on a test server if you would like to kick the tires. Thanks! L
            Hide
            Christine Doherty added a comment -

            Is there any news on this one? Stanford would like to have this very much. It would solve our dilemma with students timing out in the Tests & Quizzes tool.

            Show
            Christine Doherty added a comment - Is there any news on this one? Stanford would like to have this very much. It would solve our dilemma with students timing out in the Tests & Quizzes tool.
            Hide
            Lance Speelmon added a comment -

            FYI - We have had this in production since January 1 and while not perfect, it is working well and serving the need. L

            Show
            Lance Speelmon added a comment - FYI - We have had this in production since January 1 and while not perfect, it is working well and serving the need. L
            Hide
            Peter A. Knoop added a comment -

            Let's aim to get this merged to trunk, once the bug/sub-task SAK-12473 is addressed.

            Show
            Peter A. Knoop added a comment - Let's aim to get this merged to trunk, once the bug/sub-task SAK-12473 is addressed.
            Hide
            Peter A. Knoop added a comment -

            Lance, do you still have the test server running with this on it? Could you post a screen shoot of what the UI looks like for someone encountering the time-out? Thanks.

            Show
            Peter A. Knoop added a comment - Lance, do you still have the test server running with this on it? Could you post a screen shoot of what the UI looks like for someone encountering the time-out? Thanks.
            Hide
            Peter A. Knoop added a comment -

            Colin, do you think you or someone else on Fluid could review this anytime soon, as Ian requested? Thanks.

            Show
            Peter A. Knoop added a comment - Colin, do you think you or someone else on Fluid could review this anytime soon, as Ian requested? Thanks.
            Hide
            Colin Clark added a comment -

            I think someone on Fluid can probably have a peek at it. Do we have a working instance of it somewhere?

            Show
            Colin Clark added a comment - I think someone on Fluid can probably have a peek at it. Do we have a working instance of it somewhere?
            Hide
            Peter A. Knoop added a comment -

            Lance (or someone from IU), is there a place one can look at this feature, or can you attach a screen shot to this issue? Thanks.

            Show
            Peter A. Knoop added a comment - Lance (or someone from IU), is there a place one can look at this feature, or can you attach a screen shot to this issue? Thanks.
            Hide
            Jon Gorrono added a comment -

            Can't vote on closed issues so.....bump

            Also, if needed, UCD can try to patch this in to one of our test tags so that Colin can take a look... We'd rather wait for a merge than to continue to rely on a patch, though.

            Show
            Jon Gorrono added a comment - Can't vote on closed issues so..... bump Also, if needed, UCD can try to patch this in to one of our test tags so that Colin can take a look... We'd rather wait for a merge than to continue to rely on a patch, though.
            Hide
            Peter A. Knoop added a comment -

            I'm re-opening this issue as the branch never seems to have been merged to trunk (only to IU's local branches). Is the code indeed completed and working for this issue? (It looks like there is still one open sub-task.) If it is ready, can you merge it to trunk, or do you need Ian's help on that? Thanks.

            Show
            Peter A. Knoop added a comment - I'm re-opening this issue as the branch never seems to have been merged to trunk (only to IU's local branches). Is the code indeed completed and working for this issue? (It looks like there is still one open sub-task.) If it is ready, can you merge it to trunk, or do you need Ian's help on that? Thanks.
            Hide
            Peter A. Knoop added a comment -

            This branch has been abandoned in favour of the work associated with SAK-13987. See that one for further progress on this issue.

            Show
            Peter A. Knoop added a comment - This branch has been abandoned in favour of the work associated with SAK-13987 . See that one for further progress on this issue.

              People

              • Assignee:
                Joseph Rodriguez
                Reporter:
                Kristol Hancock
              • Votes:
                2 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Development

                    Git Source Code