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

Sync triggers twice #36

Closed
LordTermor opened this issue May 23, 2024 · 1 comment · Fixed by #37
Closed

Sync triggers twice #36

LordTermor opened this issue May 23, 2024 · 1 comment · Fixed by #37
Assignees
@LordTermor
Labels
bug Something isn't working Daemon Changes related to the backend

Comments

@LordTermor
Copy link
Collaborator

For whatever reason synchronization triggers twice when called via API
@LordTermor LordTermor self-assigned this May 23, 2024
@LordTermor LordTermor added bug Something isn't working Daemon Changes related to the backend labels May 23, 2024
@LordTermor
Copy link
Collaborator Author

LordTermor commented May 23, 2024 •
edited
Loading

Detaching the sync execution from controller fixes the issue

LordTermor added a commit to LordTermor/bxt that referenced this issue May 24, 2024
@LordTermor
fix(daemon): start sync detached
1288664 
Fixes anydistro#36
LordTermor added a commit to LordTermor/bxt that referenced this issue May 24, 2024
@LordTermor
fix(daemon): start sync detached
c72244c 
Fixes anydistro#36
@LordTermor LordTermor mentioned this issue May 24, 2024
Merged
@LordTermor LordTermor added this to the Initial testing (MVP) milestone May 24, 2024
@LordTermor LordTermor linked a pull request May 24, 2024 that will close this issue
Sync improvements #37
Merged
LordTermor added a commit to LordTermor/bxt that referenced this issue May 24, 2024
@LordTermor
fix(daemon): start sync detached
b9b7af7 
Fixes anydistro#36
LordTermor added a commit to LordTermor/bxt that referenced this issue May 25, 2024
@LordTermor
fix(daemon): start sync detached
fd4be51 
Fixes anydistro#36
LordTermor added a commit that referenced this issue May 25, 2024
@LordTermor
fix(daemon): start sync detached
7de1406 
Fixes #36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@LordTermor LordTermor

Labels
bug Something isn't working Daemon Changes related to the backend
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Sync improvements LordTermor/bxt
1 participant
@LordTermor