Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-31052 update Course AJAX/javascript to YUI3 and replace section_classes.js
  3. MDL-31720

Refactor course/rest.php

          Details

          • Type: Sub-task
          • Status: Closed
          • Priority: Minor
          • Resolution: Not a bug
          • Affects Version/s: 2.3
          • Fix Version/s: None
          • Component/s: Course, JavaScript
          • Labels:

            Gliffy Diagrams

              Attachments

                Activity

                Ascending order - Click to sort in descending order
                Hide
                Permalink
                salvetore Michael de Raadt added a comment - - edited

                Thanks for working on this, Ruslan.

                It would be nice to see at least a brief description of the work you are doing.

                Show
                salvetore Michael de Raadt added a comment - - edited Thanks for working on this, Ruslan. It would be nice to see at least a brief description of the work you are doing.
                Hide
                Permalink
                kabalin Ruslan Kabalin added a comment -

                It would be nice to see at least a brief description of the work you are doing.

                Replacing code duplicates, making it 'AJAX_SCRIPT'. Not much really, it done in fact, will push it soon along with MDL-31263

                Show
                kabalin Ruslan Kabalin added a comment - It would be nice to see at least a brief description of the work you are doing. Replacing code duplicates, making it 'AJAX_SCRIPT'. Not much really, it done in fact, will push it soon along with MDL-31263
                Hide
                Permalink
                kabalin Ruslan Kabalin added a comment -

                Ready to be reviewed

                Show
                kabalin Ruslan Kabalin added a comment - Ready to be reviewed
                Hide
                Permalink
                poltawski Dan Poltawski added a comment -

                If this specific issue needs to be reviewed please can we have the commit in here. If it is all to be reviewed in the parent task please only move that one issue to waiting for review/integration since it makes senesce for them one done in one chunk.

                Show
                poltawski Dan Poltawski added a comment - If this specific issue needs to be reviewed please can we have the commit in here. If it is all to be reviewed in the parent task please only move that one issue to waiting for review/integration since it makes senesce for them one done in one chunk.
                Hide
                Permalink
                kabalin Ruslan Kabalin added a comment -

                > If it is all to be reviewed in the parent task please only move that one issue to waiting for review/integration since it makes senesce for them one done in one chunk.
                I see, Dan.

                Show
                kabalin Ruslan Kabalin added a comment - > If it is all to be reviewed in the parent task please only move that one issue to waiting for review/integration since it makes senesce for them one done in one chunk. I see, Dan.
                Hide
                Permalink
                poltawski Dan Poltawski added a comment -

                Closing this as I think its covered by the other bugs.

                Show
                poltawski Dan Poltawski added a comment - Closing this as I think its covered by the other bugs.

                  People

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

                    Dates

                    • Created:
                      Updated:
                      Resolved: