search

mitodl / micromasters

Portal for learners and course teams to access MITx Micromasters® programs
https://mm.mit.edu
BSD 3-Clause "New" or "Revised" License
30 stars 17 forks source link

broken link to MITx Online Dashboard #5134

Closed pdpinch closed 2 years ago

pdpinch commented 2 years ago

Steps to Reproduce

  1. Log in to RC
  2. Authorize yourself for the 14.750x exam:
    1. link your account to rc.mitxonline.mit.edu
    2. enroll in 14.750x
    3. pay for 14.750x (if you have trouble with the test Visa account on CyberSource, I found success by using the admin to change the status of my order to fulfilled)
    4. using the admin, create a passing grade for yourself
    5. you should be authorized for the exam now
  3. Load the dashboard
  4. You should see the message "you can access your exam through your MITx Online Dashboad"
  5. click the link

Expected Behavior

Actual Behavior

If it requires a lot of effort (like, adding yet another setting) to make this link work, I suggest keeping things simple. Just remove the anchor tag and leave the text. This message was originally added to compensate for the fact that the prior link, to register for the course, was single use only. Since that's no longer the case, we don't really need this link any more.

pdpinch commented 2 years ago

Here is the relevant code: https://github.com/mitodl/micromasters/blob/c2965d691a91616a6d2a0af7ae19ab0a08e1f169/static/js/components/dashboard/courses/StatusMessages.js#L318

Configuring the dev environment to duplicate these conditions is will take much longer than the code modification. In the interests of time, I think it's OK to merge this change and we will test it on RC.

umarmughal824 commented 2 years ago

@pdpinch After performing all the steps mentioned in the PR description, I found out that we have set up the environment variable MITXONLINE_BASE_URL to https://courses-qa.mitxonline.mit.edu/. But that should be set to https://rc.mitxonline.mit.edu/ and for the dashboard link, I have created the PR.