View Issue Details

IDProjectCategoryView StatusLast Update
0007888Talerwallet-corepublic2023-09-23 13:09
ReporterFlorian Dold Assigned Tosebasjm Resolution deadline
ImplementerReviewer 
Release signed byDeployed by 
VerifierCommunicator 
Data protection authority notification requiredNo  
CSIRT notification requiredNo  
Estimated effort (hours)Actual effort (hours)0 
Fast-trackNo
Work packagenone
PrioritynormalSeverityfeatureReproducibilityhave not tried
Status closedResolutionno change required 
Target Version0.9.3Fixed in Version0.9.3 
Summary0007888: add currency to transaction-state-transition notification
DescriptionThis allows the UIs to make more fine-grained requests to update the list of transactions.
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

Florian Dold

2023-07-24 08:49

manager   ~0020363

Trivial to implement, but I want to finish the DB work first.

sebasjm

2023-08-23 14:05

developer   ~0020432

trivial but there are some places where the present of the currency is not guaranteed, for example a purchase that there is no downloaded info. (pay-merchant.ts)
Also the amount of places where this need to be modified is high (322 lines)
One more reason, the currency is not enough if we are going to paginate the tx list (it may be one tx is not in the view)

If the UI is showing some tx list it already knows the txId being shown, so the request to listen for tx update can be done by a list of txId. This is possible since the notification comes with txId and can be used for the fine-grained filter

Issue History

Date ModifiedUsernameFieldChange
2023-07-17 13:21Florian DoldNew Issue
2023-07-17 13:21Florian DoldStatusnew => assigned
2023-07-17 13:21Florian DoldAssigned To => Florian Dold
2023-07-24 08:49Florian DoldNote Added: 0020363
2023-07-24 09:40Christian GrothoffSeverityminor => feature
2023-08-03 18:46Florian DoldAssigned ToFlorian Dold => sebasjm
2023-08-23 14:05sebasjmNote Added: 0020432
2023-08-23 14:06sebasjmStatusassigned => informing
2023-08-23 14:06sebasjmResolutionopen => no change required
2023-09-23 13:07Christian GrothoffFixed in Version => 0.9.3
2023-09-23 13:09Christian GrothoffStatusinforming => closed