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(overlay): avoid calling push in iterable object to avoid error lo… #2088

Conversation

alejandrofernandezcanton
Copy link
Contributor

…ading overlays

What I did

Change .push in iterator to avoid error loading overlays. This only happens in old chromium version but we support old versions.
Resolves #2086

@changeset-bot
Copy link

changeset-bot bot commented Oct 9, 2023

🦋 Changeset detected

Latest commit: 7cee3e7

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@lion/ui Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@CLAassistant
Copy link

CLAassistant commented Oct 9, 2023

CLA assistant check
All committers have signed the CLA.

@alejandrofernandezcanton alejandrofernandezcanton force-pushed the fix/overlays-utils-adopt-styles branch from d1a4cfc to 2edabb5 Compare October 9, 2023 14:37
@alejandrofernandezcanton alejandrofernandezcanton force-pushed the fix/overlays-utils-adopt-styles branch from 2edabb5 to 7cee3e7 Compare October 9, 2023 14:49
Copy link
Member

@gerjanvangeest gerjanvangeest left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@alejandrofernandezcanton Thanks for the fix 💪!

@gerjanvangeest gerjanvangeest merged commit b1320a9 into ing-bank:master Oct 17, 2023
4 checks passed
@alejandrofernandezcanton
Copy link
Contributor Author

@gerjanvangeest thanks for merging the fix. Could you tell me when the next release will be?

@gerjanvangeest
Copy link
Member

We normally do it on request, so i just created the release :)

@alejandrofernandezcanton
Copy link
Contributor Author

Thanks you @gerjanvangeest

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.

Error with overlays in lion/ui 0.4.X with old chromium versions
3 participants