Skip to content

Conversation

@portuu3
Copy link
Collaborator

@portuu3 portuu3 commented Oct 29, 2025

Issue tracking

Closes #3638

Context behind the change

Add the ability to complete the escrow without payouts for situations where participants didn't reach stablished criterias

How has this been tested?

Unit tests

Release plan

  • Deploy contracts

Potential risks; What to monitor; Rollback plan

NA

@portuu3 portuu3 requested a review from Dzeranov October 29, 2025 12:39
@portuu3 portuu3 self-assigned this Oct 29, 2025
@vercel
Copy link

vercel bot commented Oct 29, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
human-app Ready Ready Preview Comment Oct 29, 2025 0:44am
human-dashboard-frontend Ready Ready Preview Comment Oct 29, 2025 0:44am
staking-dashboard Ready Ready Preview Comment Oct 29, 2025 0:44am
2 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
faucet-frontend Ignored Ignored Preview Oct 29, 2025 0:44am
faucet-server Ignored Ignored Oct 29, 2025 0:44am

Copy link
Contributor

@Dzeranov Dzeranov left a comment

Choose a reason for hiding this comment

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

LGTM
Tagging @dnechay

@Dzeranov Dzeranov requested a review from dnechay October 29, 2025 12:46
@dnechay
Copy link
Collaborator

dnechay commented Oct 29, 2025

Contract changes lgtm, not sure why CVAT tests fail though

@portuu3 portuu3 merged commit 83bb3cb into contracts-v2 Oct 29, 2025
18 of 19 checks passed
@portuu3 portuu3 deleted the contracts-3638 branch October 29, 2025 12:54
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