Skip to content

fix: correct missing interpolation - #1568

Merged
alangsto merged 1 commit into
masterfrom
alangsto/fix_modal_interpolation
Aug 18, 2022
Merged

fix: correct missing interpolation#1568
alangsto merged 1 commit into
masterfrom
alangsto/fix_modal_interpolation

Conversation

@alangsto

Copy link
Copy Markdown
Contributor

Description

The Modal scss was missing interpolation for calculated styles.

Deploy Preview

Include a direct link to your changes in this PR's deploy preview here (e.g., a specific component page).

Merge Checklist

  • If your update includes visual changes, have they been reviewed by a designer? Send them a link to the Netlify deploy preview, if applicable.
  • Does your change adhere to the documented style conventions?
  • Do any prop types have missing descriptions in the Props API tables in the documentation site (check deploy preview)?
  • Were your changes tested using all available themes (see theme switcher in the header of the deploy preview, under the "Settings" icon)?
  • Were your changes tested in the example app?
  • Is there adequate test coverage for your changes?
  • Consider whether this change needs to reviewed/QA'ed for accessibility (a11y). If so, please add wittjeff and adamstankiewicz as reviewers on this PR.

Post-merge Checklist

  • Verify your changes were released to NPM at the expected version.
  • If you'd like, share your contribution in #show-and-tell.
  • 🎉 🙌 Celebrate! Thanks for your contribution.

@netlify

netlify Bot commented Aug 18, 2022

Copy link
Copy Markdown

Deploy Preview for paragon-openedx ready!

Name Link
🔨 Latest commit 03e1e36
🔍 Latest deploy log https://app.netlify.com/sites/paragon-openedx/deploys/62fe73ab5046bb00086fb009
😎 Deploy Preview https://deploy-preview-1568--paragon-openedx.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@adamstankiewicz adamstankiewicz left a comment

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.

LGTM!

@codecov

codecov Bot commented Aug 18, 2022

Copy link
Copy Markdown

Codecov Report

Merging #1568 (03e1e36) into master (b14c5cf) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1568   +/-   ##
=======================================
  Coverage   91.46%   91.46%           
=======================================
  Files         206      206           
  Lines        3490     3490           
  Branches      818      818           
=======================================
  Hits         3192     3192           
  Misses        283      283           
  Partials       15       15           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@alangsto
alangsto force-pushed the alangsto/fix_modal_interpolation branch 3 times, most recently from 27cb8bd to 6edc324 Compare August 18, 2022 17:14
@alangsto
alangsto force-pushed the alangsto/fix_modal_interpolation branch from 6edc324 to 03e1e36 Compare August 18, 2022 17:15
@alangsto
alangsto merged commit 88fe9fd into master Aug 18, 2022
@alangsto
alangsto deleted the alangsto/fix_modal_interpolation branch August 18, 2022 17:35
edx-semantic-release pushed a commit that referenced this pull request Aug 18, 2022
## [20.9.3](v20.9.2...v20.9.3) (2022-08-18)

### Bug Fixes

* correct missing interpolation ([#1568](#1568)) ([88fe9fd](88fe9fd))
@edx-semantic-release

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 20.9.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants