View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0011172 | Taler | quality checkpoint | public | 2026-03-02 19:21 | 2026-08-18 21:28 |
| Reporter | Christian Grothoff | Assigned To | Florian Dold | Resolution deadline | |
| Implementer | Reviewer | ||||
| Release signed by | Deployed by | ||||
| Verifier | Communicator | ||||
| Data protection authority notification required | No | ||||
| CSIRT notification required | No | ||||
| Estimated effort (hours) | Actual effort (hours) | 0 | |||
| Fast-track | No | ||||
| Work package | none | ||||
| Priority | normal | Severity | feature | Reproducibility | N/A |
| Status | implementing | Resolution | open | ||
| Platform | i7 | OS | Debian GNU/Linux | OS Version | squeeze |
| Product Version | git (master) | ||||
| Target Version | 1.7 | ||||
| Summary | 0011172: send notification to merchant when KYC status changes [meta] | ||||
| Description | Right now, the merchant has to go to the backend to see the change. We should trigger an automatic notification. | ||||
| Tags | No tags attached. | ||||
| ID | Summary | Status | Assigned To | Estimated effort (hours) |
|---|---|---|---|---|
| 0011242 | when multiple bank accounts are configured, the order is non-deterministic and causes extra notifications | informing | Florian Dold | |
| 0011244 | implement harness test for KYC status notifications [1h] | informing | Florian Dold | |
| This issue (hours) | ||||
| Its children (hours) | 0 | |||
| Together (hours) | 0 | |||
Nothing is missing: the plan can be finished.
| parent of | 0011242 | informing | Florian Dold | when multiple bank accounts are configured, the order is non-deterministic and causes extra notifications |
| parent of | 0011244 | informing | Florian Dold | implement harness test for KYC status notifications [1h] |
| Plan: + add one_shot_hidden BOOL column to merchant_reports + modify GET /reports to hide those entries in REST API + modify taler-merchant-report-generator to DELETE those entries instead of UPDATING next transmission time - expand one of the /kyc endpoint to return suitable report data (PDF, Text, etc.) + add trigger on KYC status change to automatically add entry to merchant_reports when KYC status changes - add new Typst template for KYC status change notifications | |
| 8f27ed6a..717a29c9 implements the notification logic. Right now, notifications should be "always on", that is if an e-mail is configured for an instance and the KYC status changes for any account, we ALWAYS send a notification. Needs QC testing => Florian. | |
| IMO this needs some *local* test first to generate the notifications for various transitions, to be able to look at them quickly. | |
| Local test exists now (0011244), next step would be to test in the staging environment by enabling it for a particular account and observing notifications. | |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2026-03-02 19:21 | Christian Grothoff | New Issue | |
| 2026-03-02 19:21 | Christian Grothoff | Status | new => assigned |
| 2026-03-02 19:21 | Christian Grothoff | Assigned To | => Christian Grothoff |
| 2026-03-02 20:47 | Christian Grothoff | Status | assigned => implementing |
| 2026-03-07 10:29 | Christian Grothoff | Note Added: 0028029 | |
| 2026-03-07 10:29 | Christian Grothoff | Status | implementing => assigned |
| 2026-03-07 10:40 | Christian Grothoff | Note Edited: 0028029 | |
| 2026-03-07 13:29 | Christian Grothoff | Note Edited: 0028029 | |
| 2026-03-07 13:30 | Christian Grothoff | Note Edited: 0028029 | |
| 2026-03-07 15:29 | Christian Grothoff | Note Added: 0028030 | |
| 2026-03-07 15:29 | Christian Grothoff | Assigned To | Christian Grothoff => Florian Dold |
| 2026-03-07 15:29 | Christian Grothoff | Category | merchant backend => quality checkpoint |
| 2026-03-07 15:29 | Christian Grothoff | Target Version | 1.8 => 1.5 |
| 2026-03-11 13:33 | Florian Dold | Note Added: 0028093 | |
| 2026-03-12 17:35 | Florian Dold | Relationship added | parent of 0011242 |
| 2026-03-12 17:36 | Florian Dold | Status | assigned => implementing |
| 2026-03-12 17:36 | Florian Dold | Summary | send notification to merchant when KYC status changes => send notification to merchant when KYC status changes [meta] |
| 2026-03-12 17:37 | Florian Dold | Relationship added | parent of 0011244 |
| 2026-03-17 11:01 | Christian Grothoff | Target Version | 1.5 => 1.6 |
| 2026-05-29 19:21 | Christian Grothoff | Target Version | 1.6 => 1.7 |
| 2026-08-18 21:28 | Florian Dold | Note Added: 0029508 |