search

Fl4reTobiasH / HDL

2 stars 0 forks source link

Usuń "subtract fee from amount" oraz zmień pozycję i opis "Send all funds" #15

Open Fl4reTobiasH opened 5 years ago

Fl4reTobiasH commented 5 years ago

Subtract fee from amount - do wyrzucenia z gui, przy opcji send all funds ta opcja będzie użyta automatycznie, a zwykły przelew nie używa tej opcji.

Send all funds, tak jak na poniższym obrazku: image

zurekpiotr commented 5 years ago

Zostawiłbym jakieś ostrzeżenie że fee zostanie pobrane z wysyłanej kwoty. trochę słabo by było jakby ktoś miał zapłacić 15USD na koncie miał 16 a automatyczne fee wyszło 2.

tendersoft-mjb commented 5 years ago

Czy przy wysyłaniu transakcji pojawia się ekran z potwierdzeniem, który zawiera wszystkie informacje?

zurekpiotr commented 5 years ago

Jest ekran z podsumowaniem transakcji i hasłem do jej potwierdzenia.

tendersoft-mjb commented 5 years ago

to jeżeli tam jest widoczna i kwota, i fee, to jakiegoś dodatkowego ostrzeżenia bym nie generował - w koncu po to jest podsumowanie, zeby wlasnie sprawdzic czy się wszystko zgadza.

Natomiast warto dodać np. pogrubienie dla 4-ech pierwszych i 4-ech ostatnich znaków adresu, na który idzie płatność - np. w aplikacji mBanku jest coś takiego dla numeru konta i jest to bardzo wygodne pod względem zachowania bezpieczeństwa transakcji.

przykład: 3J98t1WpEZ73CNmQviecrnyiWrnqRhWNLy

Fl4reTobiasH commented 5 years ago

@tendersoft-mjb To już mamy w najnowszej wersji portfela (pogrubione i pokolorowane znaki w adresie)

image

Odnośnie ostrzeżenia, myślę że jest niepotrzebne. Opcja Subtract fee byłaby aktywna wyłącznie w momencie kliknięcia "wyślij wszystko". Ta opcja służy raczej do przeniesienia salda a nie do płatności za coś. @zurekpiotr płaciłeś kiedyś za coś opcją "Send All"? :)

tendersoft-mjb commented 5 years ago

OK, przy czym to jest w historii. Adres w podsumowaniu przed potwierdzeniem transakcji, także ma kolorki? :) @Fl4reTobiasH masz rację, co do sposobu użycia tego Subtract fee - automat przy całej kwocie, a tak to samo się dolicza - tak to wszędzie działa, więc użytkownicy oczekują takiego właśnie sposobu konstuowania transakcji. Dodatkowa opcja jest zbędna, tym bardziej że zawsze to jakieś potencjalne pole do błędów.

Fl4reTobiasH commented 5 years ago

@tendersoft-mjb Racja, w podsumowaniu nie ma kolorowania. Samo podsumowanie wymaga odrębnego opracowania żeby było bardziej czytelne. Przygotuję je.