Skip to content

chore: add synchronous syncs#58

Merged
ahmad-kemsan merged 3 commits into
mainfrom
huzaif/synchronous-server-sync
Apr 7, 2026
Merged

chore: add synchronous syncs#58
ahmad-kemsan merged 3 commits into
mainfrom
huzaif/synchronous-server-sync

Conversation

@mir-huzaif

Copy link
Copy Markdown
Contributor

No description provided.

@mir-huzaif
mir-huzaif requested a review from ahmad-kemsan April 6, 2026 12:16
@ahmad-kemsan
ahmad-kemsan requested a review from Copilot April 7, 2026 08:53

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds public, blocking “sync now” APIs to force an immediate one-time server synchronization for both license and trial activations, complementing the existing periodic/background sync behavior.

Changes:

  • Introduces sync_license_activation() and sync_trial_activation() Rust wrapper functions returning LexActivatorStatus.
  • Adds new FFI bindings SyncLicenseActivation and SyncTrialActivation to the extern interface.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File Description
src/lib.rs Adds the new public synchronous sync wrapper functions and their status mappings.
src/extern_functions.rs Exposes the corresponding new C FFI functions to Rust.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/lib.rs
Comment thread src/lib.rs
Comment thread src/lib.rs
Comment thread src/lib.rs
@ahmad-kemsan
ahmad-kemsan merged commit 2317ab6 into main Apr 7, 2026
3 of 4 checks passed
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.

3 participants