-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Consider changing the data types to something less internal #12
Comments
Then, we can talk about how a |
Yeah, agree. Or even have a human readable version, like I report in the detailed section: I think there's a good case for explaining what different data types are (strings, integers, bools), the related validators and what we use them for. For some readers, this may be the first time anyone's done that for them! |
There's a greater issue there of data-types in the sheet vs data-types in the intermediate system vs data-types in json exports. Clojure/Script has a uuid data type, for example, where it is a string in the sheet and a string in the export. Similarly, dates have multiple representations along the chain. They do have predictable transformations, though. |
I think there are three audiences, with various overlapping interests in the docs: Stuff written for B and C is generally useful to A, but stuff for A less so for B and C! |
In this column, the data types are named by their internal validation function for
clojure.spec
. As these are relatively arbitratily named, I would suggest changing them out for something 1) used more widely and 2) further abstracted 3) reveals less about inner implementation.One such solution would be to use the Typescript standard, as that is becoming a very widely used type system.
the changes would look like this
optional values get added question marks.
string?
,string[]?
, etcThe text was updated successfully, but these errors were encountered: