Document Validation flow #72
Conversation
Matej-Hlatky
had a problem deploying
to
packaging
GitHub Actions
Failure
Matej-Hlatky
had a problem deploying
to
packaging
GitHub Actions
Failure
Matej-Hlatky
temporarily deployed
to
packaging
GitHub Actions
Inactive
Matej-Hlatky
temporarily deployed
to
packaging
GitHub Actions
Inactive
| }, | ||
| ), | ||
| DocumentValidationErrorState _ => const SizedBox.shrink(), | ||
| // TODO Show error in this panel - red background with message |
There was a problem hiding this comment.
Does this still need doing?
There was a problem hiding this comment.
Yes.
Don't have desings for that :/
There was a problem hiding this comment.
Nainštaloval som si posledný build z tejto branche. Overovanie podpisov funguje v poriadku. Rovnako funguje v poridaku aj podpisovanie lokálneho súboru.
Pri podpisovaní cez QR kód zostane na konci appka v "loading" stave, aj keď extension podpísaný dokument normálne dostane a úspešne dokončí podpisovnaie na webe. Nechal som appku aj minútu v tomto stave a nepohla sa ďalej. Back button ma vráti na home screen a viem appku ďalej používať. Pri podpise lokálneho súboru problém nenastáva.
Fix infinite loading state for remote signing
# Conflicts: # lib/di.config.dart # lib/ui/screens/preview_document_screen.dart
Matej-Hlatky
force-pushed
the
feature/36-document-validation
branch
from
ebb80d0 to
3fe32a4
Compare
celuchmarek
temporarily deployed
to
packaging
GitHub Actions
Inactive
|
Pridávam screenshoty, aby sme vedeli, ako to vyzerá cc @jsuchal |
|
Kukol som výsledky testovania pre overovanie podpisov. Vypadli z neho 3 veci: Fičúra je trochu skrytáZa mňa je toto dobré pretože:
Flow bol navrhnutý tak aby sa pred podpisom skontrolovali dokumenty a v prípade nejakých problémov bol používateľ upozornený, takže nejaké fatálne následky by nastať rozhodne nemali ani pri začiatočníkoch. Zatiaľ by som sa teda tomuto bodu viac nevenoval a bral to skôr ako fičurú a nie bug. Nesedela terminológiaÚprimne si už nepamätám či som niečo nekonzistentne pomenoval v prototype, je to dosť možné. Dajte si iba prosím pozor, aby sa typy podpisov v tomto flowe volali rovnako ako počas podpisovania, resp. ako je to zaužívané medzi používateľmi. Malo by to potom byť v poriadku. Chýbali podrobnostiĽudia hovorili, že by sa radi dozvedeli viac informacii v prípadoch neplatných podpisov, aby vedeli čo teda majú robiť. Na jednu stranu sme AVM nechceli veľmi komplikovať a chceli to skôr spraviť jednoduchšie na používanie, na druhej dáva zmysel, že keď človek nájde nejaký nedostatok, rád by vedeľ čo má vtedy robiť. Zároveň metodológia toho konkrétneho testovania použiteľnosti z hackathonu nebola najvhodnejšia na otestovanie toho, ako by sa používatelia reálne správali v takej situácii a čo by potrebovali. Bolo to príliš hypotetícké a odrážame sa v podstate iba od tvrdení, nie pozorovania. Môžeme (nemusíme) teda otvoriť debatu či pri neplatných, neznámych a iných ne- typoch podpisov nechceme zobrazovať aj nejaké dodatočné info o tom čo to znamená, ako sa to asi mohlo stať a čo s tým môže používateľ robiť ďalej. Toto je však na sync a trochu dizajnovania, tak si povedzte x) |
Closes #36