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: Cannot read property rules of undefined #1189

Closed
wants to merge 1 commit into from
Closed

fix: Cannot read property rules of undefined #1189

wants to merge 1 commit into from

Conversation

vtereshyn
Copy link
Contributor

Not sure if it works for you, but I guess we shouldn't come into this condition if rule is undefined

image

@vtereshyn vtereshyn changed the title fix: Cannor read property rule of undefined fix: Cannor read property rules of undefined Sep 10, 2019
@kof
Copy link
Member

kof commented Sep 10, 2019

I think the problem is somewhere before.

@vtereshyn vtereshyn changed the title fix: Cannor read property rules of undefined fix: Cannot read property rules of undefined Sep 11, 2019
matteoferigo added a commit to matteoferigo/jss that referenced this pull request Nov 6, 2019
@matteoferigo
Copy link

matteoferigo commented Nov 6, 2019

With this change, the issue will appear later (eg: rules is used even in row 199).

To fix this issue is simply needed to prevent rules update when rule instance doesn't exist anymore

@kof
Copy link
Member

kof commented Dec 4, 2019

closing this PR since it isn't mergeable, please feel free to reopen later

@kof kof closed this Dec 4, 2019
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