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-8202 Send: Recent list isn't displayed for tokens. #4043

Merged
merged 6 commits into from
Oct 16, 2024

Conversation

amuraveinik
Copy link
Contributor

@amuraveinik amuraveinik commented Oct 11, 2024

IOS-8202

Проверялось на адресе 0xE35F09B310479fFdC10Ff583349C1020295AfEcF

Для сравнения, что выводится в приложении и в экплорере

image

Тут нужно смотреть только OUT, и чтобы справа было USDT

image

else {
return nil
}

let address: String
switch interactionAddress(from: record) {
case .user(let value), .contract(let value):
Copy link
Contributor

Choose a reason for hiding this comment

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

А остальные типы не должны тут обрабатываться?
Как минимум .multiple кажется весьма распостранен в UTXO сетках

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Договорились после дейлика что не надо

https://tangem.slack.com/archives/GMXC6PP71/p1729071867870609?thread_ts=1728984100.220299&cid=GMXC6PP71

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Рассуждения такие что у нас сейчас нет токенов в UTXO сетях, которые поддерживают историю транзакций, так что даже потестить не получится.

А просто для сетей UTXO (не в токенах) сейчас все ок

tureck1y
tureck1y previously approved these changes Oct 14, 2024
@amuraveinik amuraveinik force-pushed the bugfix/IOS-8202_recent_for_tokens branch from cb03d5f to ec141f6 Compare October 16, 2024 09:54
@amuraveinik amuraveinik changed the base branch from develop to releases/5.17 October 16, 2024 09:54
@tureck1y tureck1y merged commit e436497 into releases/5.17 Oct 16, 2024
3 checks passed
@tureck1y tureck1y deleted the bugfix/IOS-8202_recent_for_tokens branch October 16, 2024 11:17
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