Skip to content

refactor: announce agreements however old they are - #734

Merged
MoonBoi9001 merged 3 commits into
mb9/reviewfrom
mb9/announce-every-agreement-regardless-of-age
Oct 8, 2026
Merged

MoonBoi9001 merged 3 commits into
mb9/reviewfrom
mb9/announce-every-agreement-regardless-of-age

Conversation

@MoonBoi9001

Copy link
Copy Markdown
Member

Dipper no longer skips announcing agreements created before 2026-08-11, the release that first sent lifecycle events: that date only spared consumers late events for old testnet agreements. Every agreement is now announced when the chain shows it accepted or ended.

Dipper skipped announcing the accept and end of agreements created before its first release
with lifecycle events, but that only spared consumers late events for old testnet agreements.
Every agreement is now announced when the chain shows it accepted or ended.
When the listener recorded an agreement's accept and end from the chain, it recorded the end
before the accept, so an older end from before the accept, such as its offer's withdrawal, was
kept. The end is now recorded again after the accept, so the real one replaces it.
A comment on the accepted-event sweep said agreements from before events existed are never
announced, which stopped being true once the date that skipped them was removed.
@MoonBoi9001
MoonBoi9001 marked this pull request as ready for review October 8, 2026 12:23
@MoonBoi9001
MoonBoi9001 merged commit 2cb3bd5 into mb9/review Oct 8, 2026
12 checks passed
@MoonBoi9001
MoonBoi9001 deleted the mb9/announce-every-agreement-regardless-of-age branch October 8, 2026 12:24
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.

1 participant