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

Update Tokenizer.js #316

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

mohsenomidi
Copy link

Fix Can not read properties undefined (reading 'push')

Fix Can not read properties undefined (reading 'push')
@mohsenomidi
Copy link
Author

Fix mui - issue 37900

@Andarist
Copy link
Collaborator

This feels like a bandaid for some deeper problem. We need to diagnose the deeper problem and we need a test case to be added to the fixing PR, one that would fail without the proposed patch.

@mohsenomidi
Copy link
Author

Thanks for the review, then we are waiting to find out the root cause and fix.

@CodingItWrong
Copy link

I experienced this error as well on 4.3.0. As a note to others, downgrading stylis to 4.2.0 fixed it for me

@Andarist
Copy link
Collaborator

@CodingItWrong please share a repro case of the problem. We can't fix the issue without being able to diagnose it

@CodingItWrong
Copy link

@Andarist Unfortunately this code is running in the context of many layers of setup done by other team members and of proprietary client code. I understand not having a repro case blocks diagnosis, but it's not something I'm able to provide.

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

3 participants