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

Pop keys from registered after calling unregister #584

Merged
merged 3 commits into from Oct 6, 2022
Merged

Pop keys from registered after calling unregister #584

merged 3 commits into from Oct 6, 2022

Conversation

luca992
Copy link
Contributor

@luca992 luca992 commented Sep 26, 2022

Otherwise this sequence:

GlobalRegistrator.register()
GlobalRegistrator.unregister()
GlobalRegistrator.register()

will throw: Failed to register. Happy DOM has already been globally registered.

capricorn86
capricorn86 previously approved these changes Oct 6, 2022
@capricorn86
Copy link
Owner

Thanks for contributing @luca992! 🙂

Sorry for not looking into this PR earlier. There has been a lot going on.

The build is failing because the commit message doesn't follow the format defined in the contribution documentation.

If you change the commit message by doing a git amend to something in style with
#584@patch: Fixes problem in the "global-registrator" package that makes it impossible to register Happy DOM globally again after unregistering it., it will manage to build.

Then I will merge and release this directly.

…kes it impossible to register Happy DOM globally again after unregistering it.
@luca992
Copy link
Contributor Author

luca992 commented Oct 6, 2022

👍 should be all good now.

@capricorn86 capricorn86 merged commit 4b79af0 into capricorn86:master Oct 6, 2022
rudywaltz pushed a commit to rudywaltz/happy-dom that referenced this pull request Oct 7, 2022
…age that makes it impossible to register Happy DOM globally again after unregistering it.
rudywaltz pushed a commit to rudywaltz/happy-dom that referenced this pull request Oct 7, 2022
…age that makes it impossible to register Happy DOM globally again after unregistering it.
rudywaltz pushed a commit to rudywaltz/happy-dom that referenced this pull request Oct 7, 2022
…age that makes it impossible to register Happy DOM globally again after unregistering it.
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.

None yet

2 participants