Skip to content

Add RegistryCreated event#258

Open
adraffy wants to merge 2 commits intodevelopfrom
feat/bet-517-registry-created
Open

Add RegistryCreated event#258
adraffy wants to merge 2 commits intodevelopfrom
feat/bet-517-registry-created

Conversation

@adraffy
Copy link
Copy Markdown
Member

@adraffy adraffy commented Mar 30, 2026

  • added event IRegistry.RegistryCreated()
  • modified PermissionedRegistry to emit event during constructor iff roles granted
  • modified UserRegistry to emit event during initialize()
  • modified WrapperRegistry to emit event during initialize()
  • added tests

Any event arguments? bytes4 interfaceId or string tag? no

@TateB
Copy link
Copy Markdown
Member

TateB commented Apr 9, 2026

@codex

@chatgpt-codex-connector
Copy link
Copy Markdown

Codex Review: Didn't find any major issues. Swish!

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@adraffy adraffy changed the base branch from main to develop April 9, 2026 21:32
@adraffy adraffy requested a review from TateB April 9, 2026 21:32
@adraffy adraffy marked this pull request as ready for review April 9, 2026 21:32
@immunefi-magnus
Copy link
Copy Markdown

🛡️ Immunefi PR Reviews

We noticed that your project isn't set up for automatic code reviews. If you'd like this PR reviewed by the Immunefi team, you can request it manually using the link below:

🔗 Send this PR in for review

Once submitted, we'll take care of assigning a reviewer and follow up here.

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.

2 participants