IOS-7438 Filecoin #824
IOS-7438 Filecoin #824
Conversation
Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
amuraveinik
requested review from
tureck1y,
dbaturin and
a team
as code owners
|
Жду довлития воллет менеджера |
Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
|
Тесты не проходят по таймауту [!] xcodebuild -showBuildSettings timed out after 4 retries with a base timeout of 3. You can override the base timeout value with the environment variable FASTLANE_XCODEBUILD_SETTINGS_TIMEOUT, and the number of retries with the environment variable FASTLANE_XCODEBUILD_SETTINGS_RETRIES Локально все ок |
| if let error = error as? JSONRPC.APIError, error.code == 1 { | ||
| return .justWithError(output: FilecoinAccountInfo(balance: 0, nonce: 0)) | ||
| } |
There was a problem hiding this comment.
А что за код такой специальный? И фоллбек на все нули для чего тут?
There was a problem hiding this comment.
На новых аккаунтах возвращается ошибка с таким кодом, после пополнения нормализуется.
Такое ожидаемое поведение
There was a problem hiding this comment.
Точнее будет сказать аккаунт создается только после пополнения. Из дора:
дополнительные условия по блокчейну – депозита или резерва нет, но аккаунт создается только после пополнения
На коиносе похожее поведение, только там возвращается пустой результат, а тут прям ошибка с кодом
There was a problem hiding this comment.
В таким случаях мы вроде как кидаем WalletError.noAccount
There was a problem hiding this comment.
Да, все так, нужно кинуть эту ошибку
|
|
||
| static func unmarshalledSignature(from originalSignature: Data, publicKey: Data, hash: Data) throws -> Data { | ||
| let signature = try Secp256k1Signature(with: originalSignature) | ||
| let unmarshalledSignature = try signature.unmarshal(with: publicKey, hash: hash) |
There was a problem hiding this comment.
А вот эта функция из либы какой-то? Просто 27, что бы вычитаем - это аообще специфика для legacy ETH подписей, по идее, это не здесь вычитать надо, а приплюсовывать, где необходимо.
There was a problem hiding this comment.
В соседнем PRе договорились это переделать и вынести в TangemSDK, так что будет учтено
#818 (comment)
amuraveinik
dismissed stale reviews from dbaturin, m3g0byt3, Balashov152, and tureck1y
via
f03ef2a
IOS-7438