Skip to content

Wait for comments to load before finishing init#206

Merged
rhansen merged 2 commits intomainfrom
rhansen-init-race
Sep 9, 2021
Merged

Wait for comments to load before finishing init#206
rhansen merged 2 commits intomainfrom
rhansen-init-race

Conversation

@rhansen
Copy link
Member

@rhansen rhansen commented Sep 7, 2021

This should fix "TypeError: Cannot read property 'data' of undefined" in localizeExistingComments() if the pad loads quickly.

Fixes #197

@tiblu Would you mind testing this? I saw #198, but I believe this is a more comprehensive fix.

@tiblu
Copy link
Contributor

tiblu commented Sep 8, 2021

@rhansen Thanks for the update!

I could NOT reproduce #197 with this PR applied.
I closed my PR in favor of this one.

This should fix "TypeError: Cannot read property 'data' of undefined"
in `localizeExistingComments()` if the pad loads quickly.
@rhansen rhansen merged commit d1d5397 into main Sep 9, 2021
@rhansen rhansen deleted the rhansen-init-race branch September 9, 2021 07:58
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.

BUG: Error when viewing Pad with comments - sometimes

2 participants