feat(contract): implement batch_pause_subscriptions emergency admin tool#546
Closed
TheCreatorNode wants to merge 11 commits into
Closed
feat(contract): implement batch_pause_subscriptions emergency admin tool#546TheCreatorNode wants to merge 11 commits into
TheCreatorNode wants to merge 11 commits into
Conversation
|
@TheCreatorNode Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits. You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀 |
TheCreatorNode
force-pushed
the
contract-53-batch-pause-subscriptions
branch
from
bd307c2 to
47f9b04
Compare
…ib.rs Restores publish_paused, publish_resumed, publish_subscription_amount_updated, publish_subscription_interval_updated, publish_merchant_withdrawal, publish_merchant_history_cleared, publish_referred, publish_admin_transferred, publish_fee_proposed, publish_fee_committed, publish_merchant_added/removed/frozen/unfrozen, publish_grace_period_proposed/committed, and publish_min_interval_updated. These were accidentally removed when adding publish_subscription_paused. cargo check passes with 0 errors.
TheCreatorNode
force-pushed
the
contract-53-batch-pause-subscriptions
branch
from
876ebf0 to
bd24349
Compare
- Add batch_pause_subscriptions(env, users) admin-only function - Add BatchSizeExceeded error variant (SiLioLabs#21) - Add publish_subscription_paused event - Max 25 items per batch to prevent ledger size issues - Skip non-existent and already-paused entries without panic - Extend TTL for already-paused entries for idempotency - Fix redundant admin auth in propose_fee/commit_fee/propose_grace_period/commit_grace_period - Fix pre-existing test syntax errors and event name assertions - Add tests: mixed inputs, non-admin panic, max size panic
…racking, and tests
TheCreatorNode
force-pushed
the
contract-53-batch-pause-subscriptions
branch
from
198d684 to
586f8f0
Compare
- Resolved all merge conflicts: - errors.rs: deduplicated error variants, kept BatchSizeExceeded=21 - events.rs: merged publish_fee_cleared and publish_subscription_auto_resumed from master - lib.rs: kept HEAD's subscribe delegation pattern, added master's merchant-decrement logic - upgrade.rs: removed nested conflict markers, took master's propose_upgrade/commit_upgrade - test.rs: updated batch_size_panics error assertions from SiLioLabs#25 to SiLioLabs#21 - stellar.ts: took master's robust nextCursor with latestLedger check - All other Rust source and snapshot files: resolved via appropriate ours/theirs - All 184 contract tests passing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #508 [CONTRACT-53]