fix : Performance optimization with targeted sync (M2-10426, M2-10465)#1065
Open
sricharan-varanasi wants to merge 19 commits intoM2-10227-local-vs-server-syncfrom
Open
fix : Performance optimization with targeted sync (M2-10426, M2-10465)#1065sricharan-varanasi wants to merge 19 commits intoM2-10227-local-vs-server-syncfrom
sricharan-varanasi wants to merge 19 commits intoM2-10227-local-vs-server-syncfrom
Conversation
sricharan-varanasi
changed the title
sricharan-varanasi
force-pushed
the
performance-targeted-sync
branch
2 times, most recently
from
52149f2 to
4d1b980
Compare
sricharan-varanasi
changed the base branch from
dev
to
M2-10227-local-vs-server-sync
…ards compatibility
… logic T_EDITOR=true git rebase --continue o "" | git rebase --continue
… feature flag - AvailableGroupEvaluator: Use legacy 'any record exists' check when flag OFF - ConstructCompletionsService: Skip cross-device auto-completion when flag OFF
- Replace full refresh with targeted sync in useStartEntity - Skip sync entirely when feature flag is disabled (matches dev behavior) - Remove duplicate sync from InProgressActivityScreen (already synced on tap)
sricharan-varanasi
force-pushed
the
performance-targeted-sync
branch
from
4d1b980 to
c53876e
Compare
Add unit and integration tests covering the feature flag behavior: - ProgressSyncService skips in-progress flows when flag disabled - Completed flows and activities still process normally - In-progress flows dispatch with full payload when flag enabled - Per-applet flag evaluation works correctly - Redux slice upsertEntityProgression contract documented
sricharan-varanasi
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📝 Description
🔗 Jira Ticket M2-10426
🔗 Jira Ticket M2-10465
Performance fix for the cross-device flow sync feature. Replaces full refresh with targeted sync when starting activities/flows.
Changes include:
useRefreshMutationwithTargetedProgressSyncServiceinuseStartEntityInProgressActivityScreen(already synced on tap)🪤 Peer Testing
['*']): Tap a flow → should open quickly without 3-5 sec delay[]): Behavior should match dev branch exactly