digipolisantwerpdocumentation / api-requirements

5 stars 12 forks source link

return array vann strings? #25

Closed eriklenaerts closed 5 years ago

eriklenaerts commented 5 years ago

Hey, volgens de spec moeten arrays ge-encapsuleert worden in een object omdat sommige frameworks er moeilijkheden met zouden hebben. Nu is m'n vraag of een simpele array van strings ook moet ge-encapsuleerd worden? Draagt niet veel bij aan de leesbaarheid, lijkt me wat overkill... example: { [ "DELIVERABLE", "ADDON", "POLICY" ] }

swagger validator is nogal streng imho ;)

Eriklommelen commented 5 years ago

Erik,

In de DA requirements staat dat array's steeds ge-encapsuleert worden.

Met vriendelijke groeten

Erik Lommelen Digipolis WSO2 ESB\WSO2 IS\ACPaaS-Team Generaal Armstrongweg 1 2020 Antwerpen Gsm: +32 499 54 99 80, Fax: +32 3 338 79 00

www.digipolis.be/

P Denk aan het milieu vooraleer je mails print - Disclaimer On 12/09/19 10:30, Erik Lenaerts wrote:

Hey, volgens de spec moeten arrays ge-encapsuleert worden in een object omdat sommige frameworks er moeilijkheden met zouden hebben. Nu is m'n vraag of een simpele array van strings ook moet ge-encapsuleerd worden? Draagt niet veel bij aan de leesbaarheid, lijkt me wat overkill... example: { [ "DELIVERABLE", "ADDON", "POLICY" ] }

swagger validator is nogal streng imho ;)

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/digipolisantwerpdocumentation/api-requirements/issues/25?email_source=notifications&email_token=ACV6NNCN6YAF6DYOP5XOXMTQJH43LA5CNFSM4IWAZ4CKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HK5NK3A, or mute the thread https://github.com/notifications/unsubscribe-auth/ACV6NNFLNNF7REE4WNWVOCDQJH43LANCNFSM4IWAZ4CA.

PeterClaes commented 5 years ago

@eriklenaerts : json bestaat vooral uit key / value objecten en om overal consequent te zijn, wordt dit dus over de hele lijn doorgetrokken.