-
Notifications
You must be signed in to change notification settings - Fork 8
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
Issues with data #49
Comments
As @strogonoff mentioned
@ronaldtse In the |
@HassanAkbar if there are no Ping @ReesePlews can we make that assumption? Thanks. |
hello @HassanAkbar i guess if the non-english term does not exist the status is "valid" but "unpublished". with the "valid" status coming from the base english term and the "unpublished" coming from no translation from the Danish group being submitted. does this understanding make sense to everyone? @ronaldtse do you think this is an acceptable understanding? Hassan, please send me a link and i will have a check. we are still working from the staging site, correct? Question: earlier when you are talking about the UUIDs and such, this does not mean there will be a change in the api (json, rdf, turtle) links, correct? |
Just for the record, we need to make sure the status attribute value is supported by spec (register or glossarist). I have not heard of "unpublished", the closest I can think of is "invalid".On 13 May 2024, at 10:14 AM, Reese Plews ***@***.***> wrote:
hello @HassanAkbar i guess if the non-english term does not exist the status is "valid" but "unpublished". with the "valid" status coming from the base english term and the "unpublished" coming from no translation from the Danish group being submitted. does this understanding make sense to everyone? @ronaldtse do you think this is an acceptable understanding?
Hassan, please send me a link and i will have a check. we are still working from the staging site, correct?
Question: earlier when you are talking about the UUIDs and such, this does not mean there will be a change in the api (json, rdf, turtle) links, correct?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: ***@***.***>
|
I think it’s the register data model in question, so valid, invalid, superseded, retired… Unpublished items can exist as part of a proposal, but that’s about it I think.On 13 May 2024, at 7:33 PM, Anton Strogonoff ***@***.***> wrote:Just for the record, we need to make sure the status attribute value is supported by spec (register or glossarist). I have not heard of "unpublished", the closest I can think of is "invalid".On 13 May 2024, at 10:14 AM, Reese Plews ***@***.***> wrote:
hello @HassanAkbar i guess if the non-english term does not exist the status is "valid" but "unpublished". with the "valid" status coming from the base english term and the "unpublished" coming from no translation from the Danish group being submitted. does this understanding make sense to everyone? @ronaldtse do you think this is an acceptable understanding?
Hassan, please send me a link and i will have a check. we are still working from the staging site, correct?
Question: earlier when you are talking about the UUIDs and such, this does not mean there will be a change in the api (json, rdf, turtle) links, correct?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: ***@***.***>
|
@ReesePlews yes that is correct, the api will not be changed just values of the ids will be updated.
@ReesePlews The issue here is that the translation exists but it does not have a status attribute given in the excel file that is being used to generate the concepts. By the explanation provided by you and @strogonoff I believe that it should be cc: @ronaldtse |
Yes @HassanAkbar I believe this is the understanding. Thanks @ReesePlews @strogonoff ! |
This concerns the
main
branch. I see there was a push yesterday, so it looks like it’s being actively worked on. In that case, here are some issues that break toolchains that work with that repository:This may be because concepts are referencing the UUID in filename, whereas some software (e.g., RegistryKit, etc.) references the
id
from within the filestatus
, which must be one of:valid
,invalid
,superseded
, orretired
. This means they are not recognized as register items by software that validates register contents more strictlyMinor issues—this does not break anything, but will cause unexpected diffs:
---
will be removed on data updates, because serializers that write register YAML treat each file as a standalone bare document (they do not allow multiple documents within one YAML file and intentionally do not support any directives)cc @HassanAkbar @ronaldtse
The text was updated successfully, but these errors were encountered: