Skip to content

Add address validator#794

Merged
mxiao-cll merged 4 commits into
mainfrom
addr
May 26, 2026
Merged

Add address validator#794
mxiao-cll merged 4 commits into
mainfrom
addr

Conversation

@mxiao-cll
Copy link
Copy Markdown
Contributor

#OPDATA-7291

Add an address validation utility, some code taken from https://github.com/smartcontractkit/external-adapters-js/blob/main/packages/composites/proof-of-reserves/src/utils/addressValidator.ts

We will be moving address validation from PoR into the protocol adapter

@mxiao-cll mxiao-cll requested a review from a team as a code owner May 25, 2026 18:07
@github-actions
Copy link
Copy Markdown
Contributor

👋 mxiao-cll, thanks for creating this pull request!

To help reviewers, please consider creating future PRs as drafts first. This allows you to self-review and make any final changes before notifying the team.

Once you're ready, you can mark it as "Ready for review" to request feedback. Thanks!

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 25, 2026

NPM Publishing labels 🏷️

🟢 This PR has valid version labels and will cause a patch bump.

Comment thread src/validation/address.ts Outdated
@mxiao-cll mxiao-cll requested a review from dskloetc May 26, 2026 13:21
dskloetc
dskloetc previously approved these changes May 26, 2026
Comment thread src/validation/address.ts Outdated
@mxiao-cll mxiao-cll requested a review from dskloetc May 26, 2026 14:16
@mxiao-cll mxiao-cll merged commit 5af192a into main May 26, 2026
15 checks passed
@mxiao-cll mxiao-cll deleted the addr branch May 26, 2026 15:07
@github-actions
Copy link
Copy Markdown
Contributor

🚀 Successfully created version bump PR: #795

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants