Skip to content

cleanup function issue #3860

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

Merged
merged 1 commit into from
May 9, 2024
Merged

Conversation

parthbt143
Copy link
Contributor

appInjector exports function named "unmount" for cleanup so changed the wrong imported and used function

appInjector exports function named "unmount" for cleanup
so changed the wrong imported and used function
Copy link

vercel bot commented May 9, 2024

@parthbt143 is attempting to deploy a commit to the Module Federation Team on Vercel.

A member of the Team first needs to authorize it.

@CLAassistant
Copy link

CLAassistant commented May 9, 2024

CLA assistant check
All committers have signed the CLA.

@ScriptedAlchemy
Copy link
Member

Thanks. There is a way to do this via a runtime plugin so this mount method is outdated but not yet replaced. I think I have a pr open for multi react app mounts that does it automatically by just importing code that uses different react major

@ScriptedAlchemy ScriptedAlchemy merged commit d86eed9 into module-federation:master May 9, 2024
1 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants