Skip to content

docs: remove outdated State Resync Event section#1970

Open
kunalworldwide wants to merge 1 commit into
open-feature:mainfrom
kunalworldwide:fix-1809
Open

docs: remove outdated State Resync Event section#1970
kunalworldwide wants to merge 1 commit into
open-feature:mainfrom
kunalworldwide:fix-1809

Conversation

@kunalworldwide
Copy link
Copy Markdown

Description

Removes the "State Resync Events" section from as it describes behavior that flagd does not actually support.

Context

Issue #1797 clarified that flagd does not handle cases where the same flag key is defined across multiple sources. The removed section incorrectly implied that when a higher-priority source deletes its version of a flag, a resync event would restore the flag definition from a lower-priority source. This is not accurate behavior.

The section being removed included:

  • The "State Resync Events" heading and all its descriptive paragraphs
  • References to flag-merge-3.svg and flag-merge-4.svg diagrams

This is a minimal documentation-only fix with no code changes.

Closes #1809

Removes the State Resync Events section from syncs.md as it describes
behavior that flagd does not actually support. Issue open-feature#1797 clarified that
flagd does not handle cases where the same flag key is defined across
multiple sources. The removed section incorrectly implied that a resync
event would restore a flag definition from a lower-priority source when
a higher-priority source deleted its version of the flag.

Closes open-feature#1809
@kunalworldwide kunalworldwide requested review from a team as code owners May 27, 2026 18:18
@netlify
Copy link
Copy Markdown

netlify Bot commented May 27, 2026

Deploy Preview for polite-licorice-3db33c ready!

Name Link
🔨 Latest commit 89ad961
🔍 Latest deploy log https://app.netlify.com/projects/polite-licorice-3db33c/deploys/6a17356d457afa0007a17eb0
😎 Deploy Preview https://deploy-preview-1970--polite-licorice-3db33c.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@dosubot dosubot Bot added the size:S This PR changes 10-29 lines, ignoring generated files. label May 27, 2026
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request removes the 'State Resync Events' section from the documentation in docs/concepts/syncs.md. I have no feedback to provide as there are no review comments to assess.

@sonarqubecloud
Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[DOC] Remove outdated Resync Event section for Sync sources

1 participant