Skip to content

Ajr video beta#3396

Draft
aresnik11 wants to merge 9 commits into
cass-gmt-1607-publishfrom
ajr-video-beta
Draft

Ajr video beta#3396
aresnik11 wants to merge 9 commits into
cass-gmt-1607-publishfrom
ajr-video-beta

Conversation

@aresnik11

Copy link
Copy Markdown
Contributor

Overview

PR Checklist

  • Related to designs:
  • Related to JIRA ticket: [ABC-123]
  • Version plan added/updated (or not needed)
  • I have run this code to verify it works
  • This PR includes unit tests for the code change
  • This PR includes testing instructions tests for the code change
  • The alpha package of this PR is passing end-to-end tests in all relevant Codecademy repositories

Testing Instructions

Don't make me tap the sign.

  1. Go to story X
  2. Do something
  3. Do that something in dark mode
  4. Check it with VO
  5. Finish and do a celebratory dance

PR Links and Envs

Repository PR Link
Monolith Monolith PR
Mono Mono PR

@nx-cloud

nx-cloud Bot commented Jul 7, 2026

Copy link
Copy Markdown

View your CI Pipeline Execution ↗ for commit 2e291a3


☁️ Nx Cloud last updated this comment at 2026-07-07 19:36:03 UTC

@codecov

codecov Bot commented Jul 7, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.15%. Comparing base (93791ee) to head (459f49c).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@                  Coverage Diff                   @@
##           cass-gmt-1607-publish    #3396   +/-   ##
======================================================
  Coverage                  90.15%   90.15%           
======================================================
  Files                        271      272    +1     
  Lines                       5700     5701    +1     
  Branches                    1920     1929    +9     
======================================================
+ Hits                        5139     5140    +1     
  Misses                       552      552           
  Partials                       9        9           
Flag Coverage Δ
pull-request 90.15% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

aresnik11 and others added 2 commits July 7, 2026 15:32
Update vidstack-vendor.css to use the webpack ~ module resolution prefix,
which tells webpack to resolve imports from node_modules. This ensures
the CSS imports are correctly resolved in both Storybook and consumer
builds like Codecademy.

Fixes module resolution error: "Can't resolve './@vidstack/react/..." when
Codecademy's webpack processes the CSS file from the gamut dist output.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant