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

#70 support component update during mount #71

Merged
merged 9 commits into from
Aug 29, 2022

Conversation

spiffytech
Copy link
Member

@spiffytech spiffytech commented Aug 6, 2022

This PR builds on the one for #65 because it touches the same unit test I already had to resolve merge conflicts for in that PR.

… have been set by the time the child is rendered
…t mount() after rendering children

# Please enter the commit message for your changes. Lines starting
# with '#' will be kept; you may remove them yourself if you want to.
# An empty message aborts the commit.
#
# Date:      Thu Jun 2 10:01:44 2022 -0400
#
# On branch 62-incorrect-mount-timing
# Changes to be committed:
#	modified:   src/index.ts
#	modified:   src/test/componentMount.tsx
#

# Please enter the commit message for your changes. Lines starting
# with '#' will be kept; you may remove them yourself if you want to.
# An empty message aborts the commit.
#
# Date:      Thu Jun 2 10:01:44 2022 -0400
#
# interactive rebase in progress; onto 38c00db
# Last commands done (4 commands done):
#    pick 60f7d36 #62: Apply element attrs before rendering children, but call component mount() after rendering children
#    pick c045be9 #62: denote fix in changelog
# Next commands to do (3 remaining commands):
#    pick 998a563 3.1.2-beta.0
#    pick 5f90a8a #62: Fix unit tests after merging in main
# You are currently rebasing branch '50-fragment-mount-timing' on '38c00db'.
#
# Changes to be committed:
#	modified:   CHANGELOG.md
#	modified:   src/index.ts
#	modified:   src/test/componentMount.tsx
#
…re all of their children had rendered

# Please enter the commit message for your changes. Lines starting
# with '#' will be kept; you may remove them yourself if you want to.
# An empty message aborts the commit.
#
# Date:      Tue Jul 12 11:03:06 2022 -0400
#
# On branch 50-fragment-mount-timing
# Changes to be committed:
#	modified:   CHANGELOG.md
#	modified:   src/index.ts
#	new file:   src/test/fragmentMountTiming.tsx
#	modified:   src/test/test.ts
#
@spiffytech spiffytech force-pushed the 70-support-recursive-mount branch 4 times, most recently from 80c31b1 to 57666c5 Compare August 6, 2022 23:28
…ler kicks an update

# Please enter the commit message for your changes. Lines starting
# with '#' will be kept; you may remove them yourself if you want to.
# An empty message aborts the commit.
#
# Date:      Sat Aug 6 19:20:31 2022 -0400
#
# On branch 70-support-recursive-mount
# Changes to be committed:
#	modified:   CHANGELOG.md
#	modified:   src/index.ts
#	modified:   src/test/componentMount.tsx
#

# Please enter the commit message for your changes. Lines starting
# with '#' will be kept; you may remove them yourself if you want to.
# An empty message aborts the commit.
#
# Date:      Sat Aug 6 19:20:31 2022 -0400
#
# On branch 70-support-recursive-mount
# Changes to be committed:
#	modified:   CHANGELOG.md
#	modified:   src/index.ts
#	modified:   src/test/componentMount.tsx
#

# Please enter the commit message for your changes. Lines starting
# with '#' will be kept; you may remove them yourself if you want to.
# An empty message aborts the commit.
#
# Date:      Sat Aug 6 19:20:31 2022 -0400
#
# On branch 70-support-recursive-mount
# Changes to be committed:
#	modified:   CHANGELOG.md
#	modified:   src/index.ts
#	modified:   src/test/componentMount.tsx
#
@spiffytech spiffytech force-pushed the 70-support-recursive-mount branch from 57666c5 to 2f2d6bb Compare August 6, 2022 23:38
@spiffytech spiffytech merged commit 0d94b81 into main Aug 29, 2022
@spiffytech spiffytech deleted the 70-support-recursive-mount branch September 10, 2022 15:16
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.

1 participant