Skip to content

Conversation

@brian-smith-tcril
Copy link
Contributor

@brian-smith-tcril brian-smith-tcril commented Jul 29, 2024

Description

PR to get a sandbox that tests using design tokens versions of frontend-build and frontend-platform without changing this MFE to use design tokens

this is to verify that it is safe to merge the design tokens support PRs to frontend-build and frontend-platform and that they do not contain any breaking changes

How Has This Been Tested?

Please describe in detail how you tested your changes.

Screenshots/sandbox (optional):

Include a link to the sandbox for design changes or screenshot for before and after. Remove this section if it's not applicable.

Before After

Merge Checklist

  • If your update includes visual changes, have they been reviewed by a designer? Send them a link to the Sandbox, if applicable.
  • Is there adequate test coverage for your changes?

Post-merge Checklist

  • Deploy the changes to prod after verifying on stage or ask @openedx/edx-infinity to do it.
  • 🎉 🙌 Celebrate! Thanks for your contribution.

@brian-smith-tcril brian-smith-tcril added the create-sandbox open-craft-grove should create a sandbox environment from this PR label Jul 29, 2024
@codecov
Copy link

codecov bot commented Jul 29, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 93.17%. Comparing base (58aa512) to head (51aadf0).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #731   +/-   ##
=======================================
  Coverage   93.17%   93.17%           
=======================================
  Files         161      161           
  Lines        3401     3401           
  Branches      924      924           
=======================================
  Hits         3169     3169           
  Misses        214      214           
  Partials       18       18           

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

@open-craft-grove
Copy link

Sandbox deployment successful 🚀
🎓 LMS
📝 Studio
ℹ️ Grove Config, Tutor Config, Tutor Requirements

@brian-smith-tcril
Copy link
Contributor Author

@open-craft-grove
Copy link

Sandbox deployment successful 🚀
🎓 LMS
📝 Studio
ℹ️ Grove Config, Tutor Config, Tutor Requirements

PR to get a sandbox that tests using design tokens versions
of frontend-build and frontend-platform without changing
this MFE to use design tokens

this is to verify that it is safe to merge the design tokens support
PRs to frontend-build and frontend-platform and that they do
not contain any breaking changes
@brian-smith-tcril brian-smith-tcril force-pushed the test-build-and-platform-updates branch from e899784 to 51aadf0 Compare January 30, 2025 17:04
@open-craft-grove
Copy link

Sandbox deployment successful 🚀
🎓 LMS
📝 Studio
ℹ️ Grove Config, Tutor Config, Tutor Requirements

@brian-smith-tcril
Copy link
Contributor Author

closing to generate a new sandbox from a new PR

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

Labels

create-sandbox open-craft-grove should create a sandbox environment from this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants