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-7807 SUI #849

Merged
merged 34 commits into from
Oct 2, 2024
Merged

IOS-7807 SUI #849

merged 34 commits into from
Oct 2, 2024

Conversation

siblockchaina
Copy link
Contributor

@siblockchaina siblockchaina commented Sep 20, 2024

Prev PR #844

@tureck1y tureck1y changed the title Ios 7807 sui Ios-7807 SUI Sep 20, 2024
@tureck1y tureck1y changed the title Ios-7807 SUI IOS-7807 SUI Sep 20, 2024
Copy link
Collaborator

@tureck1y tureck1y left a comment

Choose a reason for hiding this comment

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

В целом все ок

BlockchainSdk/Blockchains/Sui/Network/SuiResponse.swift Outdated Show resolved Hide resolved
BlockchainSdk/Blockchains/Sui/Network/SuiResponse.swift Outdated Show resolved Hide resolved
BlockchainSdk/Blockchains/Sui/SUIUtils.swift Outdated Show resolved Hide resolved
BlockchainSdk/Blockchains/Sui/SuiWalletManager.swift Outdated Show resolved Hide resolved
tureck1y
tureck1y previously approved these changes Sep 24, 2024
dbaturin
dbaturin previously approved these changes Sep 24, 2024
BlockchainSdk/Blockchains/Sui/Network/SuiResponse.swift Outdated Show resolved Hide resolved
BlockchainSdk/Blockchains/Sui/SUIUtils.swift Outdated Show resolved Hide resolved
BlockchainSdk/Blockchains/Sui/SuiTransactionBuilder.swift Outdated Show resolved Hide resolved
BlockchainSdk/Blockchains/Sui/SuiTransactionBuilder.swift Outdated Show resolved Hide resolved
@@ -97,6 +97,8 @@ struct EstimationFeeAddressFactory {
return "EQAY92urFDKejoDRdi_EfRKLGB1JkGjD8z1inj_DhgBaD0Xo"
case .koinos:
return "1C423Vbd44zjghhJR5fKJdLFS3rgVFUc9A"
case .sui:
return ""
Copy link
Contributor

Choose a reason for hiding this comment

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

сюда бы задачку что бы не забыть )

BlockchainSdk/Blockchains/Sui/SuiTransactionBuilder.swift Outdated Show resolved Hide resolved
BlockchainSdk/Blockchains/Sui/SuiTransactionBuilder.swift Outdated Show resolved Hide resolved
BlockchainSdk/Blockchains/Sui/SuiWalletManager.swift Outdated Show resolved Hide resolved
BlockchainSdk/Blockchains/Sui/SuiWalletManager.swift Outdated Show resolved Hide resolved
BlockchainSdk/Common/API/TestnetAPINodeInfoProvider.swift Outdated Show resolved Hide resolved
@Balashov152
Copy link
Contributor

Вообще по код стайлу хорошо бы форматер нам как то заюзать, или хотя бы прогонять его раз в месяц
или на CI сделать его, но он тогда комитить будет, не факт что удобно будет

@tureck1y
Copy link
Collaborator

мы же будем переносить бсдк в репу приложения, форматер тогда сможем юзать

Balashov152
Balashov152 previously approved these changes Sep 27, 2024
Andoran90
Andoran90 previously approved these changes Sep 30, 2024
@tureck1y tureck1y merged commit 52567a8 into develop Oct 2, 2024
1 check failed
@tureck1y tureck1y deleted the IOS-7807_sui branch October 2, 2024 17:01
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.

5 participants