Skip to content

feat: Upgrade Button plugin import in learner dash - #25

Merged
mshet-sonata-pixel merged 10 commits into
frontend-basefrom
SUBS-800-11
Jul 31, 2026
Merged

feat: Upgrade Button plugin import in learner dash#25
mshet-sonata-pixel merged 10 commits into
frontend-basefrom
SUBS-800-11

Conversation

@mshet-sonata-pixel

@mshet-sonata-pixel mshet-sonata-pixel commented Jul 29, 2026

Copy link
Copy Markdown

Ticket - https://2u-internal.atlassian.net/browse/SUBS-801

Here the Upgrade Button plugin migrated from frontend-plugin-learner-dashboard into learner dashboard.

To Test:
Clone the Repo,
Be in this branch
Be in learner-dashboard url
you see the upgrade button at course card action slot


image

@mshet-sonata-pixel mshet-sonata-pixel changed the title feat: Upgrade Button plugin import in learner dash draft: Upgrade Button plugin import in learner dash Jul 29, 2026
@mshet-sonata-pixel mshet-sonata-pixel changed the title draft: Upgrade Button plugin import in learner dash feat: Upgrade Button plugin import in learner dash Jul 29, 2026
@mshet-sonata-pixel mshet-sonata-pixel changed the title draft: Upgrade Button plugin import in learner dash feat: Upgrade Button plugin import in learner dash Jul 29, 2026
Comment thread package.json Outdated
"tsc-alias": "^1.8.16",
"turbo": "^2.8.20"
"turbo": "^2.8.20",
"@edx/react-unit-test-utils": "^4.0.0"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This package installs a version of frontend-platform as a dependency. So we end up with both frontend-base and frontend-platform in the package-lock.json file. We'll need to find a way to write the UpgradeButtonWrapper.test.jsx tests without the @edx/react-unit-test-utils package. This most likely means abandoning snapshot based testing and using @testing-library/react like we do for most of our other unit tests.

For context: the snapshot pattern with @edx/react-unit-test-utils is outdated. The repository has been archived as of October 2nd, 2025. So the package doesn't have future support (or current support technically as it requires frontend-platform).

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure will fix it

@mshet-sonata-pixel
mshet-sonata-pixel merged commit 230a31c into frontend-base Jul 31, 2026
3 checks passed
@mshet-sonata-pixel
mshet-sonata-pixel deleted the SUBS-800-11 branch July 31, 2026 13:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants