IOS-7967 Enenrgy WEB EVM/X #3975
Conversation
|
Пока что билдится не будет, не подключен BSDK |
| @@ -28,7 +28,7 @@ extension SupportedBlockchains { | |||
| /// Blockchains which don't include in supported blockchains by default | |||
| static var testableIDs: Set<String> { | |||
| // Here version isn't important because we take only coinId | |||
| return Set(SupportedBlockchains(version: .v1).testableBlockchains().map { $0.coinId }) | |||
| return Set(SupportedBlockchains(version: .v1).testableBlockchains().map { $0.networkId }) | |||
There was a problem hiding this comment.
Поправьте меня если я не прав, но кажется что тут изначально должно было быть networkId.
Я столкнулся с тем что в разных сетях (energy-web-chain и energy-web-x) coin id может быть одинаковый (energy-web-token), и на бета сборке в supported blockchain будет отображаться energy-web-token.
А в Manage Tokens в свою очередь отобразится только одна сеть
There was a problem hiding this comment.
https://api.tangem-tech.com/v1/coins?networkIds=energy-web-chain
{
"imageHost": "https://s3.eu-central-1.amazonaws.com/tangem.api/coins/",
"ts": 1728034490106,
"coins": [
{
"id": "energy-web-token",
"name": "Energy Web",
"symbol": "EWT",
"active": true,
"networks": [
{
"networkId": "energy-web-chain",
"exchangeable": false
}
]
}
],
"total": 1
}https://api.tangem-tech.com/v1/coins?networkIds=energy-web-x
{
"imageHost": "https://s3.eu-central-1.amazonaws.com/tangem.api/coins/",
"ts": 1728034667245,
"coins": [
{
"id": "energy-web-token",
"name": "Energy Web",
"symbol": "EWT",
"active": true,
"networks": [
{
"networkId": "energy-web-x",
"exchangeable": false
}
]
}
],
"total": 1
}
There was a problem hiding this comment.
ммм, по большому счет было без разницы, с учетом того, что coinId уникальный, вообще надо проверить по коду не оторвет ли где-то еще повторяющийся айдишник
There was a problem hiding this comment.
кажется оторвет в сортировке группировке, в менедж токенс, в онбординге раскрытие трескается по coinId
There was a problem hiding this comment.
Я посмотрел, сейчас с бинансом такая же история и вроде как ок, но надо провести ревизию кода, заведу задачу
There was a problem hiding this comment.
| @@ -55,15 +55,18 @@ struct SupportedBlockchains { | |||
| } | |||
|
|
|||
| let betaTestingBlockchains = FeatureStorage().supportedBlockchainsIds.compactMap { id in | |||
| testableBlockchains().first { $0.coinId == id } | |||
| testableBlockchains().first { $0.networkId == id } | |||
There was a problem hiding this comment.
Тут соответственно так же
|
Нужно будет еще бампнуть версию BSDK |
|
похоже я тебе конфликт сделал синком релиза, пофикси плиз и вольем |
| @@ -140,6 +138,9 @@ struct SupportedBlockchains { | |||
| .internetComputer, | |||
| .sei(testnet: false), | |||
| .sui(curve: ed25519Curve(for: version), testnet: false), | |||
| .energyWebEVM(testnet: false), | |||
| .energyWebX(curve: ed25519Curve(for: version)), | |||
| .core(testnet: false), | |||
There was a problem hiding this comment.
Тогглы включил, как договорились на дейлике
IOS-7967
Соответствующий PR в BSDK