Skip to content
This repository has been archived by the owner on May 4, 2022. It is now read-only.

Make null message data optional #674

Open
apetro opened this issue Jan 24, 2018 · 0 comments
Open

Make null message data optional #674

apetro opened this issue Jan 24, 2018 · 0 comments

Comments

@apetro
Copy link
Contributor

apetro commented Jan 24, 2018

For all data in messages.json, where data is permissibly null, make the data outright optional so that omitting it has the same effect as if it were explicitly included and null (or whatever sensible default).

Thereby allowing convention over configuration: rather than explicitly saying a bunch of defaults about a message, don't have to configure these fields, just accept default convention.

Thereby making the data that we do configure more meaningful, since what's stated about any given message would only be what's non-default, interesting about it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant