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

senderAddress #72

Merged
merged 3 commits into from
Dec 3, 2014
Merged

senderAddress #72

merged 3 commits into from
Dec 3, 2014

Conversation

fsb4000
Copy link
Contributor

@fsb4000 fsb4000 commented Dec 1, 2014

Не для merge! Нужны доработки
В подробностях транзакции показываются адреса от кого получена транзакция.
Было:
before
Стало:
now

Нерешённые вопросы:

  1. Может быть, если у транзакции много входов, то показывать не все адреса? Если да, то какое число считать "много входов"?
  2. Возможно стоит убрать повторяющиеся адреса? Если да, то убирать ли повторяющиеся "неизвестно" ?(Например, у транзакции 3 входа, удалось получить из pubkey только 1 адрес, стоит ли писать 2 раза "неизвестно"?)
  3. В качестве адреса показывать алиас из адресной книги, если такой адрес в ней есть.

@fsb4000
Copy link
Contributor Author

fsb4000 commented Dec 2, 2014

Вроде получилось решить 3.
after "add label to sender address"

@0xDEADFACE
Copy link
Contributor

Electrum показывает в деталях все входящие и исходящие адреса, а рядом с ними суммы. Но надо ли оно.

@fsb4000
Copy link
Contributor Author

fsb4000 commented Dec 3, 2014

Может не надо:)
Изначально, просто хотелось видеть от кого пришла транзакция с помощью qt кошелька, без использования сторонних blockexplorer
если такие полосы прокрутки не пугают, то вроде правка готова
полоса прокрутки

CryptoManiac added a commit that referenced this pull request Dec 3, 2014
@CryptoManiac CryptoManiac merged commit b7f0854 into novacoin-project:master Dec 3, 2014
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.

None yet

3 participants