Skip to content

Comments

Description and Metadata URL to Pool#348

Merged
Yunusabdul38 merged 1 commit intoWeb3Novalabs:mainfrom
Oshioke-Salaki:main
Feb 20, 2026
Merged

Description and Metadata URL to Pool#348
Yunusabdul38 merged 1 commit intoWeb3Novalabs:mainfrom
Oshioke-Salaki:main

Conversation

@Oshioke-Salaki
Copy link
Contributor

No description provided.

@vercel
Copy link

vercel bot commented Feb 20, 2026

@Oshioke-Salaki is attempting to deploy a commit to the shola's projects Team on Vercel.

A member of the Team first needs to authorize it.

@drips-wave
Copy link

drips-wave bot commented Feb 20, 2026

Hey @Oshioke-Salaki! 👋 It looks like this PR isn't linked to any issue.

If this PR is for one of the issues assigned to you as part of a Wave, please link it to ensure your contribution is tracked properly. You can do this by adding a keyword to the PR description (e.g., Closes #123), or by clicking a button below:

Issue Title
#305 Add Description and Metadata URL to Pool Link to this issue

ℹ️ Learn more about linking PRs to issues

- Add  and  to Pool struct
- Update create_pool() to accept description (max 256 bytes) and
  metadata_url (max 512 bytes) parameters with input validation
- Extend PoolCreatedEvent to emit metadata_url for off-chain indexers
- Update all tests to pass the new parameters
Copy link
Collaborator

@Yunusabdul38 Yunusabdul38 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Yunusabdul38 Yunusabdul38 merged commit bbe88f3 into Web3Novalabs:main Feb 20, 2026
1 of 2 checks passed
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 Description and Metadata URL to Pool

2 participants