Uploaded image for project: 'Moodle app'
  1. Moodle app
  2. MOBILE-3286

Strip path from url when using a site url fails

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Waiting for testing
    • Priority: Minor
    • Resolution: Unresolved
    • Affects Version/s: 3.8.0
    • Fix Version/s: None
    • Component/s: Log in screen
    • Labels:
    • Testing Instructions:
      Hide
      1. In a browser, login to any moodle site.
      2. Once logged in, copy the url in the browser and use it to login in the app.
      3. The app should recognize the site, even if the url was not the root of the domain.
      4. Test the same workflow using different urls (site login url, course page, etc.)
      5. Sites that are not installed on the root path (like example.org/mymoodle/) should also work for known urls, which are: login page, home, dashboard, course and page activity.
      Show
      In a browser, login to any moodle site. Once logged in, copy the url in the browser and use it to login in the app. The app should recognize the site, even if the url was not the root of the domain. Test the same workflow using different urls (site login url, course page, etc.) Sites that are not installed on the root path (like example.org/mymoodle/) should also work for known urls, which are: login page, home, dashboard, course and page activity.
    • Affected Branches:
      MOODLE_38_STABLE
    • Pull Master Branch:
    • Sprint:
      Moodle App 3.8.1

      Description

      The site login page does not work if the url introduced is not the root of a moodle installation, for example a url such as this would fail: https://school.moodledemo.net/my/

      Given that introducing the correct url is one problem users are facing, an improvement would be to have a fallback where the path is removed from the url for a second attempt. Doing this, it would be possible to give simple instructions such as "copy the url from the browser and paste it into the app".

        Attachments

          Activity

            People

            • Assignee:
              noeldemartin Noel De Martin
              Reporter:
              noeldemartin Noel De Martin
              Peer reviewer:
              Dani Palou
              Integrator:
              Juan Leyva
              Tester:
              Isabel Renedo Rouco
              Participants:
              Component watchers:
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated: