View Issue Details

IDProjectCategoryView StatusLast Update
0007065Anastasisreducer specpublic2022-09-26 18:50
ReporterFlorian Dold Assigned Tosebasjm Due Date
PrioritynormalSeverityminorReproducibilityhave not tried
Status closedResolutionfixed 
Product VersionGit master 
Target Version0.3.0Fixed in Version0.3.0 
Summary0007065: review types of challenge feedback
DescriptionThe possible list of challenge feedback seems excessive. I'm already removing "hint" in favor of "details".

But is there a reason to ever allow an arbitrary "body"? We can always add this in the future, but I don't see any reason to let the reducer handle this. A provider returning some response we don't understand should just be considered an error.
TagsNo tags attached.

Activities

Florian Dold

2021-11-03 11:24

manager   ~0018453

The "authentication-timeout" state is also rather unclear. The documentation makes it sound like it should be "authentication-pending"? What's an example where this response can happen?

Florian Dold

2021-11-03 11:31

manager   ~0018454

"external-instructions" is way too generic. Why isn't "redirect" an external instruction? The "state" here really should be "auth-iban" or something.

Christian Grothoff

2022-03-10 03:22

manager   ~0018789

Latest reducer API has cleaned up the challenge types a lot. Please let me know if you see anything else amiss -- and update the Typescript implementation to match the latest C edition...

Florian Dold

2022-04-14 19:25

manager   ~0018880

The challenge feedback in both reducers is now cleaned up, anastasis-webui doesn't render everything properly yet.

Christian Grothoff

2022-08-24 10:55

manager   ~0019033

unknown challenge feedback status {"state":"incorrect-answer"} -- incorrectly rendered by webui.

Christian Grothoff

2022-08-24 11:04

manager   ~0019034

SM: Please try the various failure modes and make sure the webui handles/renders those nicely...

sebasjm

2022-08-26 16:09

developer   ~0019063

Added a message for every "ChallengeFeedbackStatus" value. There are two places where the feedback is shown:
1.- the policy overview
2.- the challenge detail

for [1] messages are defined here
https://git.taler.net/wallet-core.git/tree/packages/anastasis-webui/src/pages/home/ChallengeOverviewScreen.tsx#n34
I'm attaching "selection_229" as an example of a policy with challenges in every different state

for [2] messages are defined here
https://git.taler.net/wallet-core.git/tree/packages/anastasis-webui/src/pages/home/SolveScreen.tsx#n32
I'm attaching "selection_230" where it can be seeing that the feedback is shown as a notification, sometimes as an error (red) or info (blue)
Selection_230.png (30,303 bytes)   
Selection_230.png (30,303 bytes)   
Selection_229.png (111,340 bytes)   
Selection_229.png (111,340 bytes)   

sebasjm

2022-08-26 16:10

developer   ~0019064

at cf894f1dd309e48d8be380c56175219027c84fb7

Issue History

Date ModifiedUsernameFieldChange
2021-11-03 11:18Florian DoldNew Issue
2021-11-03 11:24Florian DoldNote Added: 0018453
2021-11-03 11:31Florian DoldNote Added: 0018454
2021-12-31 11:38Christian GrothoffAssigned To => Christian Grothoff
2021-12-31 11:38Christian GrothoffStatusnew => assigned
2022-03-10 03:21Christian GrothoffAssigned ToChristian Grothoff => Florian Dold
2022-03-10 03:22Christian GrothoffNote Added: 0018789
2022-04-14 19:25Florian DoldNote Added: 0018880
2022-04-14 19:26Christian GrothoffProduct Version => Git master
2022-04-14 19:26Christian GrothoffTarget Version => 0.3.0
2022-08-24 10:55Christian GrothoffNote Added: 0019033
2022-08-24 11:03Christian GrothoffAssigned ToFlorian Dold => sebasjm
2022-08-24 11:04Christian GrothoffNote Added: 0019034
2022-08-26 16:09sebasjmNote Added: 0019063
2022-08-26 16:09sebasjmFile Added: Selection_230.png
2022-08-26 16:09sebasjmFile Added: Selection_229.png
2022-08-26 16:10sebasjmStatusassigned => resolved
2022-08-26 16:10sebasjmResolutionopen => fixed
2022-08-26 16:10sebasjmNote Added: 0019064
2022-09-26 18:49Christian GrothoffFixed in Version => 0.3.0
2022-09-26 18:50Christian GrothoffStatusresolved => closed