openedx / frontend-platform

A framework for Open edX micro-frontend applications.
http://openedx.github.io/frontend-platform
GNU Affero General Public License v3.0
32 stars 64 forks source link

feat: updating dependency to react-redux@8.1.1 #549

Closed pedromartello closed 1 year ago

pedromartello commented 1 year ago

Updating dependencies to the latest react-redux@8.1.1

These changes support OEP-65 and the related work in the repository frontend-app-shell which uses the Piral framework to federate MFEs.

Piral is incopatible with the earlier version of react-redux@7.2.9 where the Redux store was not available during component rendering.

openedx-webhooks commented 1 year ago

Thanks for the pull request, @pedromartello! Please note that it may take us up to several weeks or months to complete a review and merge your PR.

Feel free to add as much of the following information to the ticket as you can:

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.

Please let us know once your PR is ready for our review and all tests are green.

codecov[bot] commented 1 year ago

Codecov Report

Patch coverage has no change and project coverage change: +0.03% :tada:

Comparison is base (26001c9) 83.05% compared to head (fefcef9) 83.08%. Report is 4 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #549 +/- ## ========================================== + Coverage 83.05% 83.08% +0.03% ========================================== Files 40 40 Lines 1062 1064 +2 Branches 195 194 -1 ========================================== + Hits 882 884 +2 Misses 168 168 Partials 12 12 ``` [see 2 files with indirect coverage changes](https://app.codecov.io/gh/openedx/frontend-platform/pull/549/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=openedx)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

openedx-webhooks commented 1 year ago

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