Skip to content

feat: add RV dynamic slot to course card action slot - #23

Merged
MaxFrank13 merged 1 commit into
release-ulmofrom
mfrank/add-dynamic-slot-for-ugprade-button-cta
Jul 27, 2026
Merged

feat: add RV dynamic slot to course card action slot#23
MaxFrank13 merged 1 commit into
release-ulmofrom
mfrank/add-dynamic-slot-for-ugprade-button-cta

Conversation

@MaxFrank13

@MaxFrank13 MaxFrank13 commented Jul 27, 2026

Copy link
Copy Markdown
Member

Summary

Adds the DynamicExperienceSlot wrapper to the CourseCardActionSlot. The DynamicExperienceSlot has an ErrorBoundary to render the default 2U UpgradeButton when RV's interface errors out. This matches the way it's been done in the advertisements repository for the UpgradeModal (here).

Related ticket / issue

Related documentation / decision records

Reviewer guidance

This will need to be tested in stage with the rest of the RV script set up in order to confirm that it is working correctly. We do have the UpgradeModal implementation that we can at least cross check with.

To run locally:

  • pull down branch to local environment
  • npm run start
  • You should see the UpgradeButton render as the DynamicExperienceSlot will error out

Screenshots

No visual changes.

Checklist

  • PR title is meaningful and describes the change clearly
  • Summary of what changed
  • Link to the related ticket or issue
  • Link to related documentation and/or decision records when applicable
  • Provided local setup and testing instructions
  • Screenshots for UI changes, if applicable

Copilot AI review requested due to automatic review settings July 27, 2026 14:50

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds a Revenue Velocity (RV) dynamic experience widget as the default content for the learner dashboard’s course card action plugin slot, with an error boundary fallback to the existing 2U upgrade CTA to keep the UI functional if the RV integration fails.

Changes:

  • Wraps CourseCardActionSlot default content in ErrorBoundary to provide a resilient fallback.
  • Renders DynamicExperienceSlot in the course card action slot using the appropriate placement and tenant identifiers.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@MaxFrank13
MaxFrank13 merged commit da4d422 into release-ulmo Jul 27, 2026
4 checks passed
@MaxFrank13
MaxFrank13 deleted the mfrank/add-dynamic-slot-for-ugprade-button-cta branch July 27, 2026 18:49
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.

3 participants