View Issue Details

IDProjectCategoryView StatusLast Update
0009038Talermerchant backendpublic2026-09-19 14:40
Reportersebasjm Assigned ToFlorian Dold 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
PrioritynormalSeverityfeatureReproducibilityalways
Status informingResolutionfixed 
Product Version0.13 
Target Version1.6Fixed in Version1.6 
Summary0009038: fix category CRUD api
Descriptiongetting category by ID doesn't bring the list of products correctly
the implementation doesn't include product description, it should by the API docs

updating should also be tested
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

Christian Grothoff

2024-08-04 20:33

manager   ~0022908

I've fixed the select_category implementation (in C), and simplified the spec to NOT return description/description_i18n -- if the SPA needs those, it can fetch them individually from the product_id anyway.

Re-assigning to sebasjm for testing with the SPA...

sebasjm

2024-08-07 15:33

developer   ~0022942

curl 'http://merchant.taler.test:1180/private/categories/15' -X 'PATCH' --data-raw '{"name":"barato","name_i18n":{},"products":[{"product_id":"beer"}],"id":"15"}'

can't change the lists of products of the category

sebasjm

2024-08-07 15:34

developer   ~0022943

i suggest to mark it for post-1.0

we can add/remove categories from the product and in this screen (updating the category) I can limit to showing instead of editing

Christian Grothoff

2024-08-07 16:04

manager   ~0022946

I'm not sure what you are talking about. Is it that when the user views a category, they cannot edit the products there? That's kind-of intentional, they should select the product and change the category with the product.

I guess it might be useful to be able to do such a change in both views (and with the category view add/remove products to the category), but indeed the API wasn't designed to support that (not with that PATCH on category).

The SPA _could_, however, simply use the PATCH on the *product(s)* to add/remove a product from a category. So this is IMO a client-side thing -- the SPA should just use a different endpoint (and possibly might have to first to a GET on the product and then a PATCH).

Anyway, if I understand correctly, I'd be happy to see this as a post-1.0 feature...

Florian Dold

2026-08-27 07:41

manager   ~0029630

Merchant API seems fine, and the UI for products/categories has been overhauled in the ng merchant webui.

Issue History

Date ModifiedUsernameFieldChange
2024-08-02 14:13sebasjmNew Issue
2024-08-02 14:13sebasjmStatusnew => assigned
2024-08-02 14:13sebasjmAssigned To => sebasjm
2024-08-04 20:15Christian GrothoffAssigned Tosebasjm => Christian Grothoff
2024-08-04 20:33Christian GrothoffNote Added: 0022908
2024-08-04 20:33Christian GrothoffAssigned ToChristian Grothoff => sebasjm
2024-08-07 15:33sebasjmNote Added: 0022942
2024-08-07 15:33sebasjmAssigned Tosebasjm => Christian Grothoff
2024-08-07 15:34sebasjmStatusassigned => implementing
2024-08-07 15:34sebasjmNote Added: 0022943
2024-08-07 16:04Christian GrothoffNote Added: 0022946
2024-08-07 16:04Christian GrothoffAssigned ToChristian Grothoff =>
2024-08-07 16:04Christian GrothoffSeveritymajor => feature
2024-08-07 16:04Christian GrothoffTarget Version0.13 => post-1.0
2024-11-12 22:56Christian GrothoffTarget Versionpost-1.0 => 1.5
2024-12-07 21:56Christian GrothoffAssigned To => Christian Grothoff
2024-12-07 21:56Christian GrothoffStatusimplementing => assigned
2024-12-22 22:16Christian GrothoffAssigned ToChristian Grothoff =>
2024-12-22 22:16Christian GrothoffStatusassigned => implementing
2025-01-02 11:57Christian GrothoffPriorityhigh => normal
2025-03-22 13:09Christian GrothoffCategorymechant backend => merchant backend
2025-10-22 19:40Christian GrothoffStatusimplementing => acknowledged
2025-10-23 10:57Christian GrothoffTarget Version1.5 => post-2.0
2026-08-27 07:41Florian DoldAssigned To => Florian Dold
2026-08-27 07:41Florian DoldStatusacknowledged => informing
2026-08-27 07:41Florian DoldResolutionopen => fixed
2026-08-27 07:41Florian DoldNote Added: 0029630
2026-09-19 14:38Christian GrothoffTarget Versionpost-2.0 => 1.6
2026-09-19 14:40Christian GrothoffFixed in Version => 1.6