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

IOS-8857 [Staking] Add support for compiling with multiple signatures #49

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

fedorov-d
Copy link

Добавил в плюсовый код метод который собирает все подписи/ключи. Пробросил его через несколько интерфейсов. Также починил тесты

@@ -108,7 +108,7 @@ class TheOpenNetworkTests: XCTestCase {
let privateKeyData = Data(hexString: "c054900a527538c1b4325688a421c0469b171c29f23a62da216e90b0df2412ee")!

let jettonTransfer = TheOpenNetworkJettonTransfer.with {
$0.jettonAmount = 500 * 1000 * 1000
$0.jettonAmount = Data(hexString: "1dcd6500")! // 500 * 1000 * 1000
Copy link
Author

Choose a reason for hiding this comment

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

фикс тестов

@@ -6,7 +6,7 @@ import Foundation

extension Data {
/// Initializes `Data` with a hex string representation.
init?(hexString: String) {
public init?(hexString: String) {
Copy link
Author

Choose a reason for hiding this comment

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

в этом файле также фиксы тестов

@fedorov-d fedorov-d requested review from Yoggam1 and tureck1y January 16, 2025 15:07
Copy link

@dbaturin dbaturin left a comment

Choose a reason for hiding this comment

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

А как раньше мы подписывали транзакции в Кардано? Не через WC, видимо?

@fedorov-d
Copy link
Author

А как раньше мы подписывали транзакции в Кардано? Не через WC, видимо?

в обычной не-стейкинг транзакции достаточно одной подписи, одна добавлялась без проблем, доработка как раз позволяет добавлять сразу несколько

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.

3 participants