search

papinet / papiNet-API

papiNet is a global paper, forest products and bioproducts industry e-Business initiative.
http://www.papinet.org/
Apache License 2.0
9 stars 3 forks source link

Property UOM in version 2.0.0 #102

Closed larsolofsson closed 9 months ago

larsolofsson commented 10 months ago

papiNet has a rule that the property names should start with a lower case character. The name of the property UOM starts with a upper case character. The name should be changed to uOM.

patricekrakow commented 10 months ago

Strictly speaking, if you follow the lowerCamelCase, I think it should be uom. The full name would be unitOfMeasure. And using the acronym UOM as not-first-word, should then look like this quantityUom.

The rules are: "lowerCamelCase names capitalize the first letter of each word, except the first which is always lowercase, even if it's an acronym." (source: https://dart.dev/effective-dart/style)

patricekrakow commented 10 months ago

We decided to go for a strict lowerCamelCase.

patricekrakow commented 9 months ago

in papiNet-API.PROPOSAL.yaml, see commit 342490a.

patricekrakow commented 9 months ago

We closed this issue with @larsolofsson, @bengtwentus and @MrNordenberg.