Skip to content

feat: add event coverage for important state changes#148

Open
Agbasimere wants to merge 1 commit into
HubDApp:mainfrom
Agbasimere:codex/add-event-coverage-state-changes
Open

feat: add event coverage for important state changes#148
Agbasimere wants to merge 1 commit into
HubDApp:mainfrom
Agbasimere:codex/add-event-coverage-state-changes

Conversation

@Agbasimere
Copy link
Copy Markdown

Closes: #134

Summary

  • add missing event/error/type plumbing for important state changes across archive/reactivate, ownership transfer, moderation, fee consumption, and settings updates
  • enrich event payloads with project identifiers, caller/admin context, timestamps, and changed fields that are useful for indexers
  • replace the outdated event test suite with focused coverage that verifies event topics and payload payloads for the key state transitions

What Changed

  • added typed events for:
    • project archive
    • project reactivate
    • ownership transfer
    • fee consumed
    • review reported
    • review hidden
    • review restored
    • fee config updates
    • minimum project age updates
  • expanded fee events to include token/operation metadata
  • added missing archive/moderation-related contract errors
  • added archived to Project state and implemented archive/reactivate registry methods
  • updated verification and fee flows so fee consumption can emit explicit events
  • refreshed test helpers/fixtures to match the current registration/update parameter shapes
  • rewrote src/tests/events.rs to validate event topics and payloads for the important indexer-facing mutations

Notes

  • tests were intentionally not run in this pass
  • branch: codex/add-event-coverage-state-changes
  • commit: 020ece8

@drips-wave
Copy link
Copy Markdown

drips-wave Bot commented Jun 2, 2026

@Agbasimere 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! 🚀

Learn more about application limits

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.

Add Event Coverage for All Important State Changes

1 participant