mitodl / edx-platform

The Open edX platform, the software that powers edX!
http://open.edx.org/
GNU Affero General Public License v3.0
6 stars 1 forks source link

fix: remove slashmark (#287) #288

Closed pdpinch closed 2 years ago

pdpinch commented 2 years ago

cherry-picked from edx-platform/master https://github.com/openedx/edx-platform/commit/cfdbb4c951eb25a1e3253d0e9ec5ec441c71ec15

Co-authored-by: connorhaugh chaugh21@amherst.edu

Related Ticket:

https://github.com/mitodl/mitxpro/issues/2332

Description

This PR cherry-picks one of the fixes merged in master in the Download Report section on Instructor Dashboard where a double slash(//) in Course Blocks API call was ending in a 404 in the backend.

Supporting information

There were some other PRs(https://github.com/openedx/edx-platform/pull/29699 & https://github.com/openedx/edx-platform/pull/29742) opened up to fix the issue at hand, but we are going with the one that has already been merged in master.

Testing instructions

pdpinch commented 2 years ago

I think if we rebase this it will clear the lint error.

pdpinch commented 2 years ago

I was hoping that the upstream PR would merge soon, but I think we've run out of time.