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

Ruling event needed on the Arbitrator side for the subgraph #520

Closed
Tracked by #518
jaybuidl opened this issue Jan 26, 2023 · 0 comments · Fixed by #543
Closed
Tracked by #518

Ruling event needed on the Arbitrator side for the subgraph #520

jaybuidl opened this issue Jan 26, 2023 · 0 comments · Fixed by #543
Assignees
Labels
Compatibility: ABI change 🗯 Smart contract ABI is changing. Package: Contracts Court smart contracts Package: Subgraph Court subgraph
Milestone

Comments

@jaybuidl
Copy link
Member

jaybuidl commented Jan 26, 2023

That’s okay, we don’t do DRY.

Alternative solution: implement some kind of discovery of the arbitrables at the graph level to avoid redeploying the subgraph every time there is a new arbitrable. This solution is doable but quite inconvenient.

@jaybuidl jaybuidl mentioned this issue Jan 26, 2023
33 tasks
@jaybuidl jaybuidl added Compatibility: ABI change 🗯 Smart contract ABI is changing. Package: Contracts Court smart contracts Package: Subgraph Court subgraph labels Jan 26, 2023
@jaybuidl jaybuidl added this to the alpha-1 milestone Jan 26, 2023
@jaybuidl jaybuidl self-assigned this Jan 26, 2023
@jaybuidl jaybuidl linked a pull request Jan 27, 2023 that will close this issue
@jaybuidl jaybuidl closed this as completed Feb 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Compatibility: ABI change 🗯 Smart contract ABI is changing. Package: Contracts Court smart contracts Package: Subgraph Court subgraph
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant