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

Have Appdb state at startup #263

Closed
stumitchell opened this issue May 14, 2020 · 2 comments
Closed

Have Appdb state at startup #263

stumitchell opened this issue May 14, 2020 · 2 comments
Labels
bug size:large

Comments

@stumitchell
Copy link

Currently when I wish to look at appdb and press ctrl-h I see the following
Screenshot 2020-05-14 at 21 00 17

The only way to get state is to trigger another event.
@superstructor superstructor added this to the v1.1.0 milestone Mar 29, 2021
@superstructor superstructor mentioned this issue Mar 29, 2021
Closed
@superstructor superstructor mentioned this issue May 25, 2021
Open
@superstructor superstructor removed this from the v1.3.0 milestone Jun 7, 2022
kimo-k added a commit that referenced this issue Aug 18, 2023
@kimo-k
Repro #263
9454a3c
kimo-k added a commit that referenced this issue Nov 30, 2023
@kimo-k
Dispatch an init event in the client to fix 10x trace state (opt-in)
3dba8ff 
Workaround for #263
kimo-k added a commit that referenced this issue Nov 30, 2023
@kimo-k
Dispatch an init event in the client to fix 10x trace state (opt-in)
0714488 
Workaround for #263
@kimo-k
Copy link
Contributor

kimo-k commented Dec 1, 2023 •
edited
Loading

I added a workaround for now. If you include the closure-define day8.re-frame-10x.init-event? true, then 10x will dispatch an event in the client re-frame which should cause the 10x UI to sync up.

See how the example project does it:

re-frame-10x/examples/todomvc/shadow-cljs.edn

Line 32 in 0714488

day8.re-frame-10x.init-event? true}

kimo-k added a commit to day8/re-frame that referenced this issue Dec 7, 2023
@kimo-k
Trace the completion of a dispatch-sync with a new :sync op-type
b364fbd 
For day8/re-frame-10x#263
kimo-k added a commit to day8/re-frame that referenced this issue Dec 7, 2023
@kimo-k
Trace the completion of a dispatch-sync with a new :sync op-type
81e8f30 
For day8/re-frame-10x#263
@kimo-k kimo-k closed this as completed in 343df85 Dec 7, 2023
kimo-k added a commit that referenced this issue Dec 7, 2023
@kimo-k
Revert "Dispatch an init event in the client to fix 10x trace state (…
edb71e8 
…opt-in)"

This reverts commit 0714488.

A workaround that didn't work very well. Making progress on a real
solution.

See #263
@kimo-k
Copy link
Contributor

kimo-k commented Dec 7, 2023

Now dispatch-sync emits a special trace to let 10x know that it has finished. You can upgrade your project to re-frame 1.4.1 to make this work. I think this particular bug is solved, but I'm still on the lookout for related bugs.

kimo-k added a commit that referenced this issue Dec 7, 2023
@kimo-k
Revert "Dispatch an init event in the client to fix 10x trace state (…
99fca6b 
…opt-in)"

This reverts commit 0714488.

A workaround that didn't work very well. Making progress on a real
solution.

See #263
kimo-k added a commit that referenced this issue Dec 7, 2023
@kimo-k
Count a new :sync trace op-type as a match terminator
485bffa 
Fixes #263.
Fixes #151.

Client must be using re-frame 1.4.1 or higher, for these fixes to take effect.
kimo-k added a commit that referenced this issue Dec 7, 2023
@kimo-k
Revert "Dispatch an init event in the client to fix 10x trace state (…
faf32c4 
…opt-in)"

This reverts commit 0714488.

A workaround that didn't work very well. Making progress on a real
solution.

See #263
kimo-k added a commit that referenced this issue Dec 7, 2023
@kimo-k
Count a new :sync trace op-type as a match terminator
7b483be 
Fixes #263.
Fixes #151.
Fixes #154.

Client must be using re-frame 1.4.1 or higher, for these fixes to take effect.
kimo-k added a commit that referenced this issue Dec 7, 2023
@kimo-k
Revert "Dispatch an init event in the client to fix 10x trace state (…
d19a51d 
…opt-in)"

This reverts commit 0714488.

A workaround that didn't work very well. Making progress on a real
solution.

See #263
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug size:large
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@superstructor @stumitchell @kimo-k