Moodle
  1. Moodle
  2. MDL-30394

Popup warning is too much information when attempting to restore.

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1.2, 2.2
    • Fix Version/s: 2.1.3
    • Component/s: Backup
    • Labels:
    • Testing Instructions:
      Hide
      1. Log in as admin
      2. Select an existing course or create new course.
      3. Backup an existing/new course (settings -> backup)
      4. Restore this course as new course
        • Start restore process (settings -> restore -> click on restore link)
        • Confirm details (Press continue)
        • Restore as a new course (In top section "Restore as a new course" select a category and click continue),
      5. Make sure it should not show any popup notification.
      Show
      Log in as admin Select an existing course or create new course. Backup an existing/new course (settings -> backup) Restore this course as new course Start restore process (settings -> restore -> click on restore link) Confirm details (Press continue) Restore as a new course (In top section "Restore as a new course" select a category and click continue), Make sure it should not show any popup notification.
    • Affected Branches:
      MOODLE_21_STABLE, MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_21_STABLE
    • Pull Master Branch:
      wip-mdl-30394
    • Rank:
      33016

      Description

      We should probably tone the popup message down to a simple note somewhere.

        Issue Links

          Activity

          Hide
          Rajesh Taneja added a comment -

          In my opinion, this information should be be explained in moodle docs.

          Show
          Rajesh Taneja added a comment - In my opinion, this information should be be explained in moodle docs.
          Hide
          David Mudrak added a comment -

          So, if I read the patch correctly, it reverts the original dialogue addition. If so, my +1 for it. Let's remove it and let us describe this more in doc.

          Show
          David Mudrak added a comment - So, if I read the patch correctly, it reverts the original dialogue addition. If so, my +1 for it. Let's remove it and let us describe this more in doc.
          Hide
          Rajesh Taneja added a comment -

          Yes David,
          It just reverts the original dialogue addition

          Show
          Rajesh Taneja added a comment - Yes David, It just reverts the original dialogue addition
          Hide
          Eloy Lafuente (stronk7) added a comment -

          bringing this in...

          Show
          Eloy Lafuente (stronk7) added a comment - bringing this in...
          Hide
          Eloy Lafuente (stronk7) added a comment -

          Integrated, thanks!

          Show
          Eloy Lafuente (stronk7) added a comment - Integrated, thanks!
          Hide
          Adrian Greeve added a comment -

          Tested. No more pop up. Thanks Rajesh.

          Show
          Adrian Greeve added a comment - Tested. No more pop up. Thanks Rajesh.
          Hide
          Eloy Lafuente (stronk7) added a comment -

          And this has landed upstream, just on time for the upcoming new releases week. Thanks for it!

          Ciao

          Show
          Eloy Lafuente (stronk7) added a comment - And this has landed upstream, just on time for the upcoming new releases week. Thanks for it! Ciao

            People

            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: