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 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

        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: