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

fix: containerShareScope should get value by shareScope key #6248

Merged
merged 2 commits into from
Apr 16, 2024

Conversation

2heal1
Copy link
Member

@2heal1 2heal1 commented Apr 16, 2024

fix: containerShareScope should get value by shareScope key

Summary

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

@github-actions github-actions bot added the release: bug fix release: bug related release(mr only) label Apr 16, 2024
@ahabhgk ahabhgk force-pushed the fix/update-federation-runtime branch from 4b35db5 to a2d1e40 Compare April 16, 2024 09:49
@2heal1 2heal1 requested a review from ahabhgk April 16, 2024 09:56
@2heal1 2heal1 changed the title fix: container should has unique name fix: containerShareScope should get value by shareScope key Apr 16, 2024
@ahabhgk ahabhgk force-pushed the fix/mf-share-scope-test branch from 3399145 to 6aeecbd Compare April 16, 2024 10:07
@ahabhgk ahabhgk merged commit 18c8cf7 into fix/update-federation-runtime Apr 16, 2024
7 checks passed
@ahabhgk ahabhgk deleted the fix/mf-share-scope-test branch April 16, 2024 10:07
ahabhgk added a commit that referenced this pull request Apr 17, 2024
* fix(core): update module federation runtime

* test(core): update module federation unit tests

* add patch

* fix

* fix

* fix: containerShareScope should get value by shareScope key (#6248)

* chore: every container should has unique name

* fix: containerShareScope should get value by shareScope key

---------

Co-authored-by: ScriptedAlchemy <zackaryjackson@bytedance.com>
Co-authored-by: ahabhgk <ahabhgk@gmail.com>
Co-authored-by: Hanric <TwoHeal@163.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release: bug fix release: bug related release(mr only)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants