Skip to content

[Fix] [Studio]: Disable auto-sync when logged out#68

Merged
Gautam25Raj merged 2 commits into
VeriWorkly:masterfrom
dicnunz:fix/disable-auto-sync-logged-out-64
May 21, 2026
Merged

[Fix] [Studio]: Disable auto-sync when logged out#68
Gautam25Raj merged 2 commits into
VeriWorkly:masterfrom
dicnunz:fix/disable-auto-sync-logged-out-64

Conversation

@dicnunz
Copy link
Copy Markdown
Contributor

@dicnunz dicnunz commented May 20, 2026

Summary

Closes #64

Tests

  • npm run test:contracts -w @veriworkly/studio
  • NEXT_PUBLIC_BACKEND_URL=http://localhost:8080/api/v1 npm run build -w @veriworkly/studio
  • git diff --check

@Gautam25Raj Gautam25Raj added bug Something isn't working frontend UI/client-side code ui/ux Design or user experience improvements labels May 21, 2026
Copy link
Copy Markdown
Collaborator

@Gautam25Raj Gautam25Raj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @dicnunz, thank you for the PR!

The core changes for disabling auto-sync when logged out look great and work perfectly.

However, we would like to keep our test suite focused and want to omit the test contract file (sync-settings.contract.test.ts) at this time.

Could you please remove the test file apps/studio/tests/contracts/sync-settings.contract.test.ts? Once that's done, we'll be ready to approve and merge this!

@Gautam25Raj Gautam25Raj changed the title Disable auto-sync when logged out [Fix] [Studio]: Disable auto-sync when logged out May 21, 2026
@Gautam25Raj Gautam25Raj merged commit 1354f68 into VeriWorkly:master May 21, 2026
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working frontend UI/client-side code ui/ux Design or user experience improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG]: Disable Auto-Sync toggle when user is logged out in settings

2 participants