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

Courts hierarchy merge - avoid collision between xDai and Ethereum court IDs on Arbitrum #522

Open
Tracked by #246
jaybuidl opened this issue Jan 26, 2023 · 0 comments
Milestone

Comments

@jaybuidl
Copy link
Member

jaybuidl commented Jan 26, 2023

Problem

Requires a translation mechanism on xDai to map the court IDs to Arbitrum court IDs without collision with the mainnet ones.

There should be only 1 General court.

Solution

There is no way to predict the v2 courtId based on v1 courtId so we must implement a mapping maintained via governance.

The conversion from v1 to v2 courtIds must take place in the xKlerosLiquidV2 adapter.
The conversion cannot take place in the ForeignGateway because the FG assumes incoming messages are v2 ones.

CourtId conversion table:

image

Consolidated hierarchy looks as follow:

image

@jaybuidl jaybuidl mentioned this issue Jan 26, 2023
33 tasks
@jaybuidl jaybuidl changed the title Courts hierarchy merge - mapping to xDai court IDs Courts hierarchy merge - avoid collision between xDai and Ethereum court IDs on Arbitrum Jan 26, 2023
@jaybuidl jaybuidl added this to the alpha-1 milestone Jan 26, 2023
@jaybuidl jaybuidl added the Package: Contracts Court smart contracts label Jan 26, 2023
@jaybuidl jaybuidl modified the milestones: testnet-beta-1, testnet-2 Aug 12, 2023
@jaybuidl jaybuidl modified the milestones: testnet-2, testnet-2.1 Oct 16, 2023
@jaybuidl jaybuidl modified the milestones: testnet-2.1, backlog Dec 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant