Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update addresses #445

Merged
merged 12 commits into from
Feb 11, 2025
Merged

update addresses #445

merged 12 commits into from
Feb 11, 2025

Conversation

dianakocsis
Copy link
Contributor

@dianakocsis dianakocsis commented Feb 6, 2025

UR v1 - doesn't support as many NFT protocols
UnsupportedProtocol - deployed if there is no support for v2 or for some nft protocols
UR v1.2 (v2 support or no v2 support) - added support for more NFT protocols
UR v2 - doesn't support NFT protocols, has v4 support

@dianakocsis dianakocsis requested a review from a team as a code owner February 6, 2025 15:33
@hensha256
Copy link
Collaborator

UnsupportedProtocol - deployed in case no v2 support

UnsupportedProtocol was also deployed for all the cases of NFT protocols not being available on some chains, even if v2 was supported!

deploy-addresses/blast.json Outdated Show resolved Hide resolved
deploy-addresses/soneium.json Outdated Show resolved Hide resolved
script/deployParameters/DeployUnichain.s.sol Outdated Show resolved Hide resolved
script/deployParameters/DeploySoneium.s.sol Outdated Show resolved Hide resolved
deploy-addresses/op-sepolia.json Outdated Show resolved Hide resolved
deploy-addresses/unichain-sepolia.json Outdated Show resolved Hide resolved
@@ -1,4 +1,3 @@
{
"UniversalRouter": "0x78e7F1502A9e4115dEbd6876E0AC4FaEBDB96880",
Copy link
Collaborator

Choose a reason for hiding this comment

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

👀 how come you removed this?
This is URV2, it supports v4

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ohh i thought u said we should remove it #445 (comment)
or should i just change it to URV2?

Copy link
Collaborator

Choose a reason for hiding this comment

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

oh yeah youre right lol

deploy-addresses/soneium.json Outdated Show resolved Hide resolved
@dianakocsis dianakocsis merged commit b122a8d into main Feb 11, 2025
5 checks passed
@dianakocsis dianakocsis deleted the update-addresses branch February 11, 2025 14:06
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