Skip to content

chore(devnet): Add support for voting new approved code hash on devnet#742

Merged
4 commits merged intomainfrom
701-task-add-support-for-voting-new-approved-code-hash-on-devnet
Jul 31, 2025
Merged

chore(devnet): Add support for voting new approved code hash on devnet#742
4 commits merged intomainfrom
701-task-add-support-for-voting-new-approved-code-hash-on-devnet

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Jul 31, 2025

Resolves #701

Tested manually that voting a new image hash works:
https://testnet.nearblocks.io/txns/4A2NTyTQPWu67Bmmf4yuUaMVdAXZsmxxxqYtYydafk7j

@ghost ghost self-assigned this Jul 31, 2025
@ghost ghost force-pushed the 701-task-add-support-for-voting-new-approved-code-hash-on-devnet branch from 5299558 to b6ffb85 Compare July 31, 2025 07:17
@ghost ghost requested a review from kevindeforth July 31, 2025 07:18
Copy link
Copy Markdown
Contributor

@kevindeforth kevindeforth left a comment

Choose a reason for hiding this comment

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

I would approve, if it wasn't for the small contract code change. I think we should have either the deref trait, or a getter, but not both at the same time.

Comment thread libs/chain-signatures/contract/src/primitives/thresholds.rs Outdated
@ghost
Copy link
Copy Markdown
Author

ghost commented Jul 31, 2025

I would approve, if it wasn't for the small contract code change. I think we should have either the deref trait, or a getter, but not both at the same time.

Fair. It requires more extensive refactoring to remove it, so I created a tracking issue. #744

I'll undo the change.

@ghost ghost marked this pull request as ready for review July 31, 2025 12:51
@ghost ghost added this pull request to the merge queue Jul 31, 2025
Merged via the queue into main with commit 34a3b2f Jul 31, 2025
7 checks passed
@ghost ghost deleted the 701-task-add-support-for-voting-new-approved-code-hash-on-devnet branch July 31, 2025 13:34
This pull request was closed.
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.

[Task] Add support for voting new approved code hash on devnet

2 participants