Skip to content
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

Add property type descriptions to JSON model #913

Merged
merged 3 commits into from
Nov 30, 2022

Conversation

tillprochaska
Copy link
Contributor

We already include schema descriptions in the JSON model, this improves consistency across types and schemata.

I’ve also converted the property type doc strings from reStructuredText to Markdown. I think the main reason for using reST is the current documentation setup is using Sphinx, but we’re mostly moving away from that and Markdown is a more common format.

@tillprochaska tillprochaska force-pushed the till/type-descriptions-json-model branch from 10becc8 to 99e17f5 Compare November 25, 2022 12:36
This is necessary as the tests rely on the type descriptions being present in the JSON model.
@tillprochaska tillprochaska force-pushed the till/type-descriptions-json-model branch from 99e17f5 to 39eaa35 Compare November 25, 2022 12:55
@tillprochaska tillprochaska mentioned this pull request Nov 25, 2022
14 tasks
@@ -183,6 +184,7 @@ def to_dict(self) -> PropertyTypeToDict:
data: PropertyTypeToDict = {
"label": gettext(self.label),
"plural": gettext(self.plural),
"description": gettext(self.docs),
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We currently don’t translate the descriptions and I’m not sure if we ever will, but it doesn’t hurt either…

@tillprochaska tillprochaska marked this pull request as ready for review November 25, 2022 13:01
@Rosencrantz Rosencrantz merged commit 424092b into main Nov 30, 2022
@pudo pudo deleted the till/type-descriptions-json-model branch February 8, 2023 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants