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
    • Rank:
      45149

      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.

        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: