You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Ah, seems this was introduced in cd8958f as we slightly changed the behavior of PaymentDetails::update there. While it previously was ~correct to interpret the return value Ok(false) as 'not found', now it really means 'was changed', i.e., 'needs repersisting'. Will see to open a PR with a fix soon.
Activity
tnull commentedon Mar 23, 2025
Ah, seems this was introduced in cd8958f as we slightly changed the behavior of
PaymentDetails::update
there. While it previously was ~correct to interpret the return valueOk(false)
as 'not found', now it really means 'was changed', i.e., 'needs repersisting'. Will see to open a PR with a fix soon.debug_assert
that would have us panic for replayedPaymentClaimed
s #511