Skip to content

Ensure that details are only synchronized for stored issues#299

Merged
nikku merged 6 commits into
mainfrom
sync-details
May 7, 2026
Merged

Ensure that details are only synchronized for stored issues#299
nikku merged 6 commits into
mainfrom
sync-details

Conversation

@nikku
Copy link
Copy Markdown
Owner

@nikku nikku commented May 7, 2026

Which issue does this PR address?

Closes #298

  • Now background sync always synchronizes issue core (it always synchronized details anyway)
  • Now comments, status and checks are only synchronized for issues in the store

nikku added 6 commits May 7, 2026 22:57
@nikku
feat(background-sync): fetch details only for updated issues
d894d28 
This ensures that issues removed due to `ignoreFilter` are not actually
synchronized.

Related to #298
@nikku
feat(background-sync): always update all issues
88b770f 
We always updated the details anyway. Skipping the issue core update
would lead to de-sync when the issue was actually outdated (remote)
and no longer valid as per `ignoreFilter`.

Related to #298
@nikku
chore(issue-filter): simplify code
8b890ce
@nikku
chore(issue-filter): add ability to create webhook filter
2c65fed
@nikku
fix(apps): don't save comments / reviews / statuses for ignored issues
faba5e2 
Closes #298
@nikku
chore(CHANGELOG): update to v0.73.3
2a5105c
@nikku nikku self-assigned this May 7, 2026
@nikku nikku merged commit 7a757f9 into main May 7, 2026
3 checks passed
@nikku nikku deleted the sync-details branch May 7, 2026 21:27
@nikku
Copy link
Copy Markdown
Owner Author

nikku commented May 7, 2026

Released fix via v0.73.3.

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

Reviewers

No reviews

Assignees

@nikku nikku

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

500 when a pull request is opened

1 participant

@nikku