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

Prevent skip change event in AltStore #6

Open
wants to merge 28 commits into
base: master
Choose a base branch
from

Conversation

davidshimjs
Copy link

Parent render -> Child render -> Child did mount -> Parent did mount

As you see, Child Component will mounted first. If It does trigger action related the Store that parent component has, Parent can't listen it. because Parent doesn't mounted yet. so I moved listening stores to componentWillMount.

@HeberLZ
Copy link

HeberLZ commented Feb 2, 2017

Hi @goatslacker , i'm having this same issue, basically changes made to the store during the initialization of the container's child component won't propagate :S

@davidshimjs
Copy link
Author

This PR has some problems. When using Server Side Rendering like Iso, Memory Leak issue can occur because only componentWillMount is called in the rendering. so you have to avoid this problem yourself.

prcolaco pushed a commit to decentrawise/alt-container that referenced this pull request Mar 7, 2024
…parser-js-0.7.33

Bump ua-parser-js from 0.7.31 to 0.7.33
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