-
Notifications
You must be signed in to change notification settings - Fork 3
Open
Description
Problem
When creating a project in the TaskManager contract, events are emitted in the following order:
ProjectManagerUpdatedProjectRolePermSetBountyCapSetProjectCreated
This causes issues for subgraph indexing and any other event consumers that expect ProjectCreated to fire first. The subgraph currently requires a workaround (ensureProjectExists()) to handle this.
Expected Behavior
ProjectCreated should be emitted before any other project-related events (ProjectManagerUpdated, ProjectRolePermSet, BountyCapSet).
Why This Matters
- Logical ordering: The entity should exist before it's configured
- Simpler subgraph code: No need for stub creation workarounds
- Better for all consumers: Any service listening to these events benefits
Suggested Fix
In the project creation function, emit ProjectCreated first, before emitting other configuration events.
Related
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels