openedx / public-engineering

General public issue repository for the Open edX engineering community
4 stars 2 forks source link

[DEPR]: MATLAB-specific handling in edx-platform #192

Open ormsbee opened 1 year ago

ormsbee commented 1 year ago

Overview

Our system currently has special handling for MATLAB problems via an old integration through xqueue that isn't really supported any longer, and appears to be broken today. Since this integration should now be done via LTI, we can remove:

This would eliminate the "MATLAB API Key" advanced course setting UI in Studio:

Screenshot 2023-05-11 at 10 08 33 AM

And the equivalent one at the ProblemBlock level:

Screenshot 2023-05-11 at 10 08 04 AM

Effect on old course content:

Proposal Date

10 May 2023

Target Ticket Acceptance Date

24 May 2023

Earliest Open edX Named Release Without This Functionality

Quince.

Since it is not currently in a working state, there's no need to offer an extra release for migration.

Rationale

  1. The current endpoint does not hook up to a working external service, and is not useful.
  2. LTI is already the accepted way to integrate MATLAB content.
  3. It will slightly simplify ProblemBlock and the LMS's XBlock runtime to remove this.
ormsbee commented 1 year ago

There was a slack thread about this as well.

kdmccormick commented 1 year ago

Just to be clear @ormsbee , would this DEPR also cover the Matlab API Key field in the problem editor, thus clearing up 2U's T&L team to remove it from the new React-based problem editor? (cc @jristau1984)

ormsbee commented 1 year ago

@kdmccormick: Yes, but to be clear: I don't think that work needs to wait for this DEPR to be accepted, since it's more of a bookkeeping DEPR–i.e. I'm pretty sure that it's been broken for years, so immediately removing the view of the broken thing doesn't negatively impact users. I'm leaving it open for a couple of weeks of comment Just In Case, and if people want to add any more information.

ormsbee commented 1 year ago

This DEPR was announced here: https://discuss.openedx.org/t/deprecation-removal-matlab-specific-handling-in-edx-platform/10069/5