Skip to content

Conversation

@sirpy
Copy link
Contributor

@sirpy sirpy commented Sep 18, 2025

Description

XDC support and small updates

Description by Korbit AI

What change is being made?

Add and wire up XDC deployment support including new interfaces, a distribution helper contract, updated UBIScheme initialization, and deployment/test scripts to accommodate multi-chain distribution and governance flows.

Why are these changes being made?

Enable XDC network deployment and multi-chain distribution capabilities, with configurable minActiveUsers and updated test/deployment plumbing to reflect the new constructor parameters and bridge flows.

Is this description stale? Ask me to generate a new description by commenting /korbit-generate-pr-description

@korbit-ai
Copy link

korbit-ai bot commented Sep 18, 2025

Korbit doesn't automatically review large (3000+ lines changed) pull requests such as this one. If you want me to review anyway, use /korbit-review.

Copy link

@sourcery-ai sourcery-ai bot left a comment

Choose a reason for hiding this comment

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

Sorry @sirpy, your pull request is larger than the review limit of 150000 diff characters

accounts: [deployerPrivateKey]
},
"development-xdc": {
...xdc
Copy link
Contributor Author

Choose a reason for hiding this comment

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

why @blueogin

Copy link
Collaborator

Choose a reason for hiding this comment

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

Because script uses network.name to save contract address in deployment.json

await releaser(torelease, network.name, "deployment", false);

Comment on lines +85 to +86
amount0Min: amount0.mul(8).div(10),
amount1Min: amount1.mul(8).div(10),
Copy link
Contributor Author

Choose a reason for hiding this comment

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

why mul 8 div 10?

Copy link
Collaborator

Choose a reason for hiding this comment

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

I set it to 80% of amountDesired, which effectively serves as a price sanity check.

@openzeppelin-code
Copy link

Xdc deploy

Generated at commit: 4a459c606e42d1e40e8e586b67070c5d6c005696

🚨 Report Summary

Severity Level Results
Contracts Critical
High
Medium
Low
Note
Total
3
5
0
16
44
68
Dependencies Critical
High
Medium
Low
Note
Total
0
0
1
0
125
126

For more details view the full report in OpenZeppelin Code Inspector

@L03TJ3 L03TJ3 self-requested a review October 3, 2025 15:15
@blueogin blueogin merged commit 0ae516f into master Oct 3, 2025
20 of 21 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.

4 participants