-
Notifications
You must be signed in to change notification settings - Fork 3
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
Word choice: field preferred over object or array #285
Comments
Also refer to top level fields as "section", e.g. the |
I think we can use field/object/array instead of section. |
Having started going through all of documentation to address this I think the following should be added to the style guide to help with this and the related language issues. (the language of my suggestions needs refined but the basic ideas are hopefully sound)
or
(don't think using object here instead of field adds anything) None of this is inconsistent with the current style guide, it's just adding further detail to remove some of the inconsistencies. |
"object" and "array" are used in their JSON sense, e.g. "array" should always be used in contexts like "Add the 'buyer' code to the "object" is useful when we're about to talk about fields within an object. To reduce ambiguity, maybe:
When referring to a subschema (i.e. Capitalized definition in the JSON schema):
|
ok your suggestions look good. Just want to flag that this is going to lead to a more substantial change to the docs, particularly https://standard.open-contracting.org/latest/en/schema/reference/#release-structure than simply swapping out words like "block" (which was where this all started in open-contracting/standard#1347) EDIT: but possibly not actually that substantial now that I start looking at it again |
Hmm, okay, let me know how substantial it's looking. We want to first focus on replacing the "block" occurrences. If it turns out to be substantial, we can postpone applying the style guide for the other words. |
It's not actually that substantial. I was concerned that the structure of https://standard.open-contracting.org/latest/en/schema/reference/#release-structure would become slightly confusing with the change of words (and therefore require a restructure) but having gone through it now I think it's fine with just a few sentence tweaks to accommodate the word swaps. |
There are still various inconsistencies in that page, e.g. "Date" is included under the "Subschema reference" heading but it's not a subschema, it's a data format. Or "Document" refers only to the different stages the |
Good observations! Can you add those (and any new ones) to open-contracting/standard#1075? |
will do, I'd forgotten there was already an issue for that page :) |
@odscjen are you happy to prepare PR to update https://ocds-standard-development-handbook.readthedocs.io/en/latest/meta/style_guide.html#data-concepts with the wording from #285 (comment)? |
As part of the PR to remove instances of "block" and "building block" decision is now to use "field" to refer to json elements rather than "object" or "array" unless referring to them as such is specifically contributing to the understanding of the sentence. See open-contracting/standard#1660 (review)
The text was updated successfully, but these errors were encountered: