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.

Sign up for GitHub

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

Jump to bottom

playingContentのウィンドウ間同期が無限ループすることがある #43

Closed
ci7lus opened this issue Feb 18, 2022 · 5 comments
Closed

playingContentのウィンドウ間同期が無限ループすることがある #43

ci7lus opened this issue Feb 18, 2022 · 5 comments
Labels
bug Something isn't working help wanted Extra attention is needed v2 v2に関連するIssue
Milestone
v2

Comments

@ci7lus
Copy link
Owner

ci7lus commented Feb 18, 2022 •
edited
Loading

ループしてるのがplayingContentかどうかもわからない、activeWindowIdの変更がループしてる気もするが
次発現したときに要確認
1c06acf
多分ここでstringifyで比較するようになったのが原因？な気がするがよくわからない、idとdurationだけ見るようにしてもいいけど、そもそもウィンドウ間ではkeyごと1秒に1回ぐらいしか更新できないようにしたほうがいいのかもしれない
@ci7lus ci7lus added bug Something isn't working help wanted Extra attention is needed labels Feb 18, 2022
@ci7lus
Copy link
Owner Author

ci7lus commented Mar 3, 2022

21f36b9
ちょっと読んでて怪しいなーとなったのでいらない機能だし削除してシンプルにしてみた

@ci7lus
Copy link
Owner Author

ci7lus commented Mar 11, 2022

なおってない

@ci7lus ci7lus added the v2 v2に関連するIssue label Mar 25, 2022
@ci7lus ci7lus added this to the v2 milestone Mar 25, 2022
@ci7lus
Copy link
Owner Author

ci7lus commented May 28, 2022

recoil-syncというのが来て直せそう、ただ破壊的な変更になってくるので現状でbeta5をリリースして適用したものをrc1とする

@ci7lus
Copy link
Owner Author

ci7lus commented Jun 1, 2022

facebookexperimental/Recoil#1824
DefaultValueの返し方がわかってなかったけどnew DefaultValue()とのことだった、言われてみれば返り値の型としてDefaultValue | unknownなんだからそのインスタンスとするのが自然、気づかなかった…

ci7lus added a commit that referenced this issue Jun 1, 2022
@ci7lus
🎨 Atomの同期構造を改善 (#43)
8dbf884
ci7lus added a commit that referenced this issue Jun 1, 2022
@ci7lus
🎨 Atomの同期構造を改善 (#43)
3ba0c47
@ci7lus ci7lus mentioned this issue Jun 1, 2022
Merged
@ci7lus
Copy link
Owner Author

ci7lus commented Jun 7, 2022

多分治った

@ci7lus ci7lus closed this as completed Jun 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working help wanted Extra attention is needed v2 v2に関連するIssue
Projects
None yet
Milestone
v2
Development

No branches or pull requests
1 participant
@ci7lus