Uploaded image for project: 'Moodle app'
  1. Moodle app
  2. MOBILE-778 META: Migrate to Ionic
  3. MOBILE-993

Participants page pull to refresh is not a real refresh

    XMLWordPrintable

    Details

    • Type: Sub-task
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0
    • Fix Version/s: 2.0
    • Component/s: Participants add-on
    • Labels:
    • Testing Instructions:
      Hide
      1. In the app, access the participants page and load all the participants so their requests are cached.
      2. In Moodle web, enrol a new user in the course.
      3. In the app, use pull to refresh and check that the new participant is shown in the list.
        • Also, you can open the console and check that the following message is written when using pull to refresh: "$mmSite: Cached element found, but it is expired".

      To test if more than one call is invalidated (infinite scroll) it is recommended that the value of mmaParticipantsListLimit is changed.

      Show
      In the app, access the participants page and load all the participants so their requests are cached. In Moodle web, enrol a new user in the course. In the app, use pull to refresh and check that the new participant is shown in the list. Also, you can open the console and check that the following message is written when using pull to refresh: "$mmSite: Cached element found, but it is expired" . To test if more than one call is invalidated (infinite scroll) it is recommended that the value of mmaParticipantsListLimit is changed.
    • Affected Branches:
      MOODLE_20_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE
    • Pull Master Branch:

      Description

      Looking at the code only, it appears that the pull to refresh does not invalidate the cache of the queries that were done before. The only thing it seems to be doing is flushing the scope variable.

      It needs to invalidate the cache and find a way to invalidate the infinite scroll queries too.

        Attachments

          Activity

            People

            Assignee:
            dpalou Dani Palou
            Reporter:
            fred Frédéric Massart
            Peer reviewer:
            Frédéric Massart Frédéric Massart
            Integrator:
            Juan Leyva Juan Leyva
            Tester:
            Juan Leyva Juan Leyva
            Participants:
            Component watchers:
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              31/Jul/15