Skip to content

refactor: split events.rs into per-domain submodules - #445

Open
Ebuka321 wants to merge 1 commit into
HubDApp:mainfrom
Ebuka321:feat/split-events-into-submodules
Open

refactor: split events.rs into per-domain submodules#445
Ebuka321 wants to merge 1 commit into
HubDApp:mainfrom
Ebuka321:feat/split-events-into-submodules

Conversation

@Ebuka321

Copy link
Copy Markdown

Summary

Split the monolithic 1956-line events.rs file into domain-specific submodules for better maintainability and readability:

  • events/project.rs – project lifecycle, claims, bookmarks, follows, endorsements
  • events/review.rs – review CRUD and moderation events
  • events/verification.rs – verification requests, renewals, disputes, assignment
  • events/fee.rs – fee configuration, payments, refunds
  • events/admin.rs – admin management, timelock, collections, pause/resume
  • events/mod.rs – public re-exports for full backward compatibility

All existing use crate::events::* imports continue to work unchanged.

Closes #344

Split the monolithic 1956-line events.rs into domain-specific submodules:
- events/project.rs: project, claim, bookmark, follow, endorsement events
- events/review.rs: review and review moderation events
- events/verification.rs: verification, renewal, dispute events
- events/fee.rs: fee-related events
- events/admin.rs: admin, timelock, collection, pause events
- events/mod.rs: re-exports all public items for backward compatibility

All existing imports via 'use crate::events::*' continue to work unchanged.
@drips-wave

drips-wave Bot commented Jul 29, 2026

Copy link
Copy Markdown

@Ebuka321 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

@Naomi-Gift

Copy link
Copy Markdown
Contributor

@Ebuka321 please resolve conflicts

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.

events.rs (1918 lines) should be split into per-domain submodules

2 participants