Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-68804

Not redirecting to the MoodleNet endpoint

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Done
    • Affects Version/s: 3.8.3, 3.9
    • Fix Version/s: None
    • Component/s: Other
    • Story Points:
      1
    • Sprint:
      Path to glory

      Description

      When a user enters their MoodleNet profile URL into either:
      a) the activity chooser pane (master) or
      b) The instance select page (38)

      The process should be to validate the profile and then redirect the user to the MoodleNet endpoint, where the MoodleNet instance will capture the site, course and section (this is a late change, adding course and section like this). Somehow, we've missed setting the endpoint in our redirects and we're not currently able to hand the user off properly.

      The endpoint is:
      lms/moodle/search

      Making a request url look like
      your.moodlenet.site/lms/moodle/search?site=xxx&course=yyy&section=z

      What I'm seeing in the current branches (based on prototype site testing) is as follows:
      Master (this is probably closer to correct):
      I see a request to https://home.next.moodle.net with site and path params correctly urlencoded and set. Here were only missing the endpoint path. We need to drop the path param as we no longer need to send this and add course and section params.

      38 (this seems quite off the mark):
      I see a request for the user's profile on MoodleNet, no params sent at all.

      So, we need to fix this in both branches by the looks of it.

        Attachments

          Activity

            People

            Assignee:
            peterdias Peter Dias
            Reporter:
            jaked Jake Dallimore
            Peer reviewer:
            Mathew May
            Participants:
            Component watchers:
            Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Time Tracking

                Estimated:
                Original Estimate - 0 minutes
                0m
                Remaining:
                Remaining Estimate - 0 minutes
                0m
                Logged:
                Time Spent - 5 hours, 45 minutes
                5h 45m