Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.4
    • Fix Version/s: 2.4
    • Component/s: Calendar
    • Labels:
    • Testing Instructions:
      Hide
      1. Login as admin
      2. Go to Navigation > Site pages > Calendar.
      3. Click the button 'Manage subscriptions'.
      4. Try importing a calendar with the URL http://www.mozilla.org/projects/calendar/caldata/AustraliaHolidays.ics , set "Poll interval" to weekly
      5. Now change "Poll interval" for this subscription to hourly and click update
      6. Make sure this "Poll interval" is set to hourly.
      Show
      Login as admin Go to Navigation > Site pages > Calendar. Click the button 'Manage subscriptions'. Try importing a calendar with the URL http://www.mozilla.org/projects/calendar/caldata/AustraliaHolidays.ics , set "Poll interval" to weekly Now change "Poll interval" for this subscription to hourly and click update Make sure this "Poll interval" is set to hourly.
    • Affected Branches:
      MOODLE_24_STABLE
    • Fixed Branches:
      MOODLE_24_STABLE
    • Pull Master Branch:
      wip-mdl-36351

      Description

      As commented by Raj in MDL-16660:

      Poll interval can't be updated. Clicking update sets poll interval to never (0)

      1. Add ical subscription with "Poll interval" to hourly
      2. Change poll to weekly, and click update. Poll is set to never.

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            Ankit Agarwal added a comment -

            Hi Raj,
            This is looking good.
            Please feel free to submit for integration.
            Thanks

            Show
            Ankit Agarwal added a comment - Hi Raj, This is looking good. Please feel free to submit for integration. Thanks
            Hide
            Rajesh Taneja added a comment -

            Thanks Ankit, pushing for integration

            Show
            Rajesh Taneja added a comment - Thanks Ankit, pushing for integration
            Hide
            Sam Hemelryk added a comment -

            Thanks Raj, this has been integrated now

            Show
            Sam Hemelryk added a comment - Thanks Raj, this has been integrated now
            Hide
            Sam Hemelryk added a comment -

            Passing tests as the QA issue will cover it.

            Show
            Sam Hemelryk added a comment - Passing tests as the QA issue will cover it.
            Hide
            Sam Hemelryk added a comment -

            (also noting I tested it anyway )

            Show
            Sam Hemelryk added a comment - (also noting I tested it anyway )
            Hide
            Dan Poltawski added a comment -

            Congratulations! Another bug solved.. only another 7330 to go, thanks for contributing to contributing to 0.8% of all bugs being fixed this week!

            ciao
            Dan

            Show
            Dan Poltawski added a comment - Congratulations! Another bug solved.. only another 7330 to go, thanks for contributing to contributing to 0.8% of all bugs being fixed this week! ciao Dan

              People

              • Votes:
                2 Vote for this issue
                Watchers:
                5 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: