Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[DEPR]: Upgrade Button #438

Open
jsnwesson opened this issue Aug 27, 2024 · 3 comments
Open

[DEPR]: Upgrade Button #438

jsnwesson opened this issue Aug 27, 2024 · 3 comments
Assignees
Labels
depr Proposal for deprecation & removal per OEP-21 needs maintainer attention Issue or PR specifically needs the attention of the maintainer.

Comments

@jsnwesson
Copy link
Contributor

jsnwesson commented Aug 27, 2024

Proposal Date

2024-08-27

Target Ticket Acceptance Date

2024-09-10

Earliest Open edX Named Release Without This Functionality

Sumac - 2024-10

Rationale

The Upgrade Button has been the most commonly requested component by the Open edX community for removal. As far as we can tell, 2U is the only organization that uses the component, which is why a complete removal of the button is the proposed approach as opposed to making it an optional component that can be injected within a PluginSlot. If the code is indeed unused, it only serves as a liability to the community and should be removed.

Removal

This is a proposal to remove the Upgrade Button component, its implementation in CourseCardActions, and any hooks logic and tracking events that are only related to "upgrading" a course.

For the sake of removing the Upgrade Button before the Sumac release, this deprecation proposal doesn't include completely removing upgrade-related information from the LearnerDashboard.Init() call to the LMS, which also means that the remaining evidence within Learner Dashboard of a course ever being upgradeable would be found in a redux selector. If this were to remain, then it would at least give the Maintaining team the time to remove the Upgrade data from the init, as any need to call LearnerDashboard.Init() more than once will have a tremendous impact on the performance of the page.

Replacement

With these removals, in CourseCardActions there would be an empty PluginSlot in place of where the Upgrade Component was implemented to allow for the component to be plugged in using Frontend Plugin Framework. This would also allow any other organization to add whatever action components they would like.

Deprecation

No response

Migration

No response

Additional Info

@jsnwesson
Discourse Post

Task List

  • [] Add PluginSlot in CourseCardActions
  • [] Remove Upgrade Button component and implementation in CourseCardActions
  • [] Remove tracking events related to course upgrade
@github-actions github-actions bot added the depr Proposal for deprecation & removal per OEP-21 label Aug 27, 2024
@jsnwesson jsnwesson self-assigned this Aug 27, 2024
@julianajlk
Copy link
Contributor

@jsnwesson hello! Is there an associated ticket for this work/which squad is removing it? Purchase squad uses this Upgrade Button and we were not aware of the removal

@jsnwesson
Copy link
Contributor Author

Hi @julianajlk ! This work is to be done by my team (Aperture). To be clear, we'll actually be keeping the Upgrade button for 2U, but it will be stored as a plugin and preserved in frontend-plugin-learner-dashboard. It is intended to work exactly as before, but the benefit is that this button doesn't exist out-of-the-box for Learner Dashboard.

@julianajlk
Copy link
Contributor

@jsnwesson Great, I misunderstood and assumed we'd need to make changes on our end to accommodate for this change. If you need any testing to be done when this goes out to production, please let us know. Thank you!

@deborahgu deborahgu added the needs maintainer attention Issue or PR specifically needs the attention of the maintainer. label Aug 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
depr Proposal for deprecation & removal per OEP-21 needs maintainer attention Issue or PR specifically needs the attention of the maintainer.
Projects
Status: In Progress
Status: Accepted
Development

No branches or pull requests

3 participants