Skip to content

Merge data layer sync info along with buttons' in order to react to data layer updates#1397

Merged
BurningAXE merged 1 commit into
release/2025.3.0from
bugfix/sync-info-view-model-does-not-auto-update
Oct 9, 2025
Merged

Merge data layer sync info along with buttons' in order to react to data layer updates#1397
BurningAXE merged 1 commit into
release/2025.3.0from
bugfix/sync-info-view-model-does-not-auto-update

Conversation

@BurningAXE
Copy link
Copy Markdown
Contributor

@BurningAXE BurningAXE commented Oct 8, 2025

JIRA ticket
Will be released in: 2025.3.0-dev

Root cause analysis (for bugfixes only)

First known affected version: version

  • Sync Info card does not update if sync button is disabled
  • More info in this discussion

Notable changes

  • Data layer flow is merged along with button flows in order to keep updating after onStart

Testing guidance

  • Describe how the reviewers can verify that issue is fixed

Additional work checklist

  • Effect on other features and security has been considered
  • Design document marked as "In development" (if applicable)
  • External (Gitbook) and internal (Confluence) Documentation is up to date (or ticket created)
  • Test cases in Testiny are up to date (or ticket created)
  • Other teams notified about the changes (if applicable)

@cla-bot cla-bot Bot added the ... label Oct 8, 2025
@BurningAXE BurningAXE requested a review from Copilot October 8, 2025 17:03
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR merges data layer sync information flow with existing button-driven sync info flows to enable reactive updates when the data layer changes. This ensures the sync information display responds to both user interactions and underlying data changes.

  • Added dataLayerDrivenSyncInfoFlow to the merge operation alongside existing button-responsive flows
  • Enables reactive sync info updates based on data layer changes rather than just button interactions

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@BurningAXE BurningAXE force-pushed the bugfix/sync-info-view-model-does-not-auto-update branch from 9ebdc2d to 96ebeef Compare October 9, 2025 08:59
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Oct 9, 2025

@BurningAXE BurningAXE marked this pull request as ready for review October 9, 2025 09:51
@BurningAXE BurningAXE requested review from a team, TristramN, alex-vt, alexandr-simprints, luhmirin-s, meladRaouf and ybourgery and removed request for a team October 9, 2025 09:51
@BurningAXE BurningAXE requested a review from Copilot October 9, 2025 10:58
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@BurningAXE BurningAXE merged commit c7eb380 into release/2025.3.0 Oct 9, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants