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

[BD-46] chore: sync alpha #2568

Merged
merged 123 commits into from
Sep 1, 2023

Conversation

monteri
Copy link
Contributor

@monteri monteri commented Aug 22, 2023

Description

Include a description of your changes here, along with a link to any relevant Jira tickets and/or Github issues.

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.

PKulkoRaccoonGang and others added 30 commits June 2, 2023 12:54
* feat: added bootstrap icons

* refactor: some refactoring

* fix: make all icons have 24x24 dimensions

* fix: update icons scripts after packages update

---------

Co-authored-by: Viktor Rusakov <[email protected]>
* docs: add copy URL and reset buttons to playground page
* chore: use Stack component and add a variable for empty state
…espective Form sets (openedx#1988)

* feat: add ability to use unlabeled Radio and Checkbox components in respective Form sets
* docs: add unlabeled control for `RadioControl` component
…eferred max width can determine when compact view occurs (openedx#2338)

* feat: add max width customizability to the stepper header's compact view

---------

Co-authored-by: Jason Wesson <[email protected]>
* docs: fixed issue feedback error
* refactor: after review
* fix: removed white background on Stepper.Header component
* refactor: after review
react-loading-skeleton defines an exports field in its package.json which looks like this:

"exports": {
    ".": {
      "types": "./dist/index.d.ts",
      "require": "./dist/index.cjs",
      "import": "./dist/index.js"
    },
    "./dist/skeleton.css": "./dist/skeleton.css"
  },

The webpack documentation states that if an exports field is defined in package.json, it replaces the default module loading behavior and any other request for a module will result in a ModuleNotFound error.

Our call to load react-loading-skeleton in Paragon looks like this:

@import "~react-loading-skeleton/dist/skeleton";

That isn’t in the list of possible exports.  This is though:

@import "~react-loading-skeleton/dist/skeleton.css";

This commit updates our import to the latter, which fixes downstream issues in MFEs that can’t figure out how to import react-loading-skeleton
* feat: extended edx typescript config

* refactor: config refactoring

* refactor: code refactoring

* build: override commitlint action to fix upstream issue with typescript

---------

Co-authored-by: Viktor Rusakov <[email protected]>
…edx#2151)

* feat: Add fixed increment scroll behavior to useOverflowScroll

* feat: eslint, tests, docs

---------

Co-authored-by: monteri <lansevermore>
* refactor: refactoring usage insights
* refactor: refactorimg Usage Insights in www
* refactor: refactoring structure
* refactor: refactoring gatsby node
* refactor: added paragon type of usage
@PKulkoRaccoonGang PKulkoRaccoonGang force-pushed the zadorozhnii/sync-alpha-v3 branch from 5e8afdc to 3ecfee6 Compare August 28, 2023 18:36
@PKulkoRaccoonGang PKulkoRaccoonGang force-pushed the zadorozhnii/sync-alpha-v3 branch from 3ecfee6 to 319f309 Compare August 29, 2023 12:49
@monteri monteri marked this pull request as ready for review August 29, 2023 14:23
@monteri
Copy link
Contributor Author

monteri commented Aug 31, 2023

Build is failing due to the skeleton.css issue

@codecov
Copy link

codecov bot commented Aug 31, 2023

Codecov Report

Patch coverage has no change and project coverage change: +0.36% 🎉

Comparison is base (8959d73) 91.58% compared to head (5537250) 91.95%.

Additional details and impacted files
@@            Coverage Diff             @@
##            alpha    #2568      +/-   ##
==========================================
+ Coverage   91.58%   91.95%   +0.36%     
==========================================
  Files         214      216       +2     
  Lines        3519     3618      +99     
  Branches      852      891      +39     
==========================================
+ Hits         3223     3327     +104     
+ Misses        288      286       -2     
+ Partials        8        5       -3     

see 20 files with indirect coverage changes

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

@monteri
Copy link
Contributor Author

monteri commented Aug 31, 2023

Build is failing due to the skeleton.css issue

Moved skeleton.css to the index.css

@adamstankiewicz adamstankiewicz merged commit ac50be8 into openedx:alpha Sep 1, 2023
8 of 9 checks passed
@openedx-webhooks
Copy link

@monteri 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future.

adamstankiewicz added a commit that referenced this pull request Sep 1, 2023
adamstankiewicz added a commit that referenced this pull request Sep 1, 2023
@edx-semantic-release
Copy link
Contributor

🎉 This PR is included in version 21.0.0-alpha.42 🎉

The release is available on:

Your semantic-release bot 📦🚀

@edx-semantic-release
Copy link
Contributor

🎉 This PR is included in version 22.0.0-alpha.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

@openedx-semantic-release-bot

🎉 This PR is included in version 23.0.0-alpha.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

PKulkoRaccoonGang pushed a commit that referenced this pull request Aug 1, 2024
PKulkoRaccoonGang pushed a commit that referenced this pull request Aug 1, 2024
PKulkoRaccoonGang pushed a commit that referenced this pull request Aug 4, 2024
PKulkoRaccoonGang pushed a commit that referenced this pull request Aug 4, 2024
PKulkoRaccoonGang pushed a commit that referenced this pull request Aug 4, 2024
PKulkoRaccoonGang pushed a commit that referenced this pull request Aug 4, 2024
PKulkoRaccoonGang pushed a commit that referenced this pull request Aug 5, 2024
PKulkoRaccoonGang pushed a commit that referenced this pull request Aug 5, 2024
@openedx-semantic-release-bot

🎉 This PR is included in version 23.0.0 🎉

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
blended PR is managed through 2U's blended developmnt program released on @alpha released
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.