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

Handle custom media layout specs. #2077

Merged
merged 3 commits into from
Oct 2, 2023
Merged

Handle custom media layout specs. #2077

merged 3 commits into from
Oct 2, 2023

Conversation

jeresig
Copy link
Member

@jeresig jeresig commented Oct 2, 2023

Summary:

We weren't taking into account custom ones, which was a problem if you wante to use a custom one!

Issue: XXX-XXXX

Test plan:

@jeresig jeresig self-assigned this Oct 2, 2023
@changeset-bot
Copy link

changeset-bot bot commented Oct 2, 2023

🦋 Changeset detected

Latest commit: f449345

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 11 packages
Name Type
@khanacademy/wonder-blocks-layout Patch
@khanacademy/wonder-blocks-birthday-picker Patch
@khanacademy/wonder-blocks-cell Patch
@khanacademy/wonder-blocks-dropdown Patch
@khanacademy/wonder-blocks-form Patch
@khanacademy/wonder-blocks-grid Patch
@khanacademy/wonder-blocks-labeled-field Patch
@khanacademy/wonder-blocks-modal Patch
@khanacademy/wonder-blocks-tooltip Patch
@khanacademy/wonder-blocks-search-field Patch
@khanacademy/wonder-blocks-popover Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@khan-actions-bot khan-actions-bot requested a review from a team October 2, 2023 13:40
@khan-actions-bot
Copy link
Contributor

khan-actions-bot commented Oct 2, 2023

Gerald

Required Reviewers
  • @Khan/wonder-blocks for changes to .changeset/dull-jobs-leave.md, packages/wonder-blocks-layout/src/components/media-layout.tsx

Don't want to be involved in this pull request? Comment #removeme and we won't notify you of further changes.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 2, 2023

Size Change: +5 B (0%)

Total Size: 90.1 kB

Filename Size Change
packages/wonder-blocks-layout/dist/es/index.js 1.88 kB +5 B (0%)
ℹ️ View Unchanged
Filename Size
packages/wonder-blocks-accordion/dist/es/index.js 55 B
packages/wonder-blocks-banner/dist/es/index.js 3.09 kB
packages/wonder-blocks-birthday-picker/dist/es/index.js 1.69 kB
packages/wonder-blocks-breadcrumbs/dist/es/index.js 1.13 kB
packages/wonder-blocks-button/dist/es/index.js 3.98 kB
packages/wonder-blocks-cell/dist/es/index.js 2.19 kB
packages/wonder-blocks-clickable/dist/es/index.js 3.21 kB
packages/wonder-blocks-color/dist/es/index.js 1.15 kB
packages/wonder-blocks-core/dist/es/index.js 3.67 kB
packages/wonder-blocks-data/dist/es/index.js 6.33 kB
packages/wonder-blocks-dropdown/dist/es/index.js 12 kB
packages/wonder-blocks-form/dist/es/index.js 5.42 kB
packages/wonder-blocks-grid/dist/es/index.js 1.36 kB
packages/wonder-blocks-i18n/dist/es/index.js 4.54 kB
packages/wonder-blocks-icon-button/dist/es/index.js 2.22 kB
packages/wonder-blocks-icon/dist/es/index.js 3.04 kB
packages/wonder-blocks-labeled-field/dist/es/index.js 72 B
packages/wonder-blocks-link/dist/es/index.js 3.03 kB
packages/wonder-blocks-modal/dist/es/index.js 5.04 kB
packages/wonder-blocks-pill/dist/es/index.js 1.03 kB
packages/wonder-blocks-popover/dist/es/index.js 4.33 kB
packages/wonder-blocks-progress-spinner/dist/es/index.js 1.51 kB
packages/wonder-blocks-search-field/dist/es/index.js 1.53 kB
packages/wonder-blocks-spacing/dist/es/index.js 158 B
packages/wonder-blocks-switch/dist/es/index.js 2.06 kB
packages/wonder-blocks-testing/dist/es/index.js 3.94 kB
packages/wonder-blocks-theming/dist/es/index.js 1.21 kB
packages/wonder-blocks-timing/dist/es/index.js 1.78 kB
packages/wonder-blocks-toolbar/dist/es/index.js 862 B
packages/wonder-blocks-tooltip/dist/es/index.js 5.05 kB
packages/wonder-blocks-typography/dist/es/index.js 1.49 kB

compressed-size-action

@github-actions
Copy link
Contributor

github-actions bot commented Oct 2, 2023

npm Snapshot: Published

🎉 Good news!! We've packaged up the latest commit from this PR (fc7e702) and published all packages with changesets to npm.

You can install the packages in webapp by running:

./services/static/dev/tools/deploy_wonder_blocks.js --tag="PR2077"

Packages can also be installed manually by running:

yarn add @khanacademy/wonder-blocks-<package-name>@PR2077

@codecov
Copy link

codecov bot commented Oct 2, 2023

Codecov Report

Merging #2077 (f449345) into main (cc6b195) will decrease coverage by 0.03%.
The diff coverage is 100.00%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2077      +/-   ##
==========================================
- Coverage   96.97%   96.94%   -0.03%     
==========================================
  Files         236      236              
  Lines       26682    26685       +3     
  Branches     2360     2313      -47     
==========================================
- Hits        25874    25869       -5     
- Misses        808      816       +8     
Files Coverage Δ
...nder-blocks-layout/src/components/media-layout.tsx 99.31% <100.00%> (+<0.01%) ⬆️

... and 1 file with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update cc6b195...f449345. Read the comment docs.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 2, 2023

A new build was pushed to Chromatic! 🚀

https://5e1bf4b385e3fb0020b7073c-fcdjmdwqtw.chromatic.com/

Chromatic results:

Metric Total
Captured snapshots 270
Tests with visual changes 1
Total stories 364
Inherited (not captured) snapshots [TurboSnap] 38
Tests on the build 308

@jeresig
Copy link
Member Author

jeresig commented Oct 2, 2023

Landing to unblock @matthewmorgan - he confirmed that this is working for him, though!

@jeresig jeresig merged commit b6fbd63 into main Oct 2, 2023
@jeresig jeresig deleted the custom-media-layout branch October 2, 2023 14:07
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