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

added ~1500 translations to Portuguese and fixed typos on previous translations #33

Merged
merged 4 commits into from
Feb 6, 2025

Conversation

dilzacampos
Copy link

There were some typos on a few terms corrected, added 1500 contributions from orcids 0000-0001-6276-958, 0000-0001-8149-365X, 0009-0001-0105-7733 and 0009-0005-0193-5224. All reviewed by 0009-0005-0193-5224.

Copy link
Contributor

@matentzn matentzn left a comment

Choose a reason for hiding this comment

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

One small request (needs to be applied in multiple places)

en pt HP:0010769 rdfs:label Pilonidal sinus Cisto pilonidal OFFICIAL orcid:0009-0005-0193-5224
en pt HP:0010770 rdfs:label Pilonidal fistula Fístula pilonidal OFFICIAL orcid:0009-0005-0193-5224
en pt HP:0010769 rdfs:label Pilonidal sinus Cisto pilonidal OFFICIAL orcid:0009-0005-0193-5224, orcid:0000-0001-6276-958
en pt HP:0010770 rdfs:label Pilonidal fistula Fístula pilonidal OFFICIAL orcid:0009-0005-0193-5224, orcid:0000-0001-6276-958
Copy link
Contributor

Choose a reason for hiding this comment

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

multiple orcids have to be tab separated like so:

Suggested change
en pt HP:0010770 rdfs:label Pilonidal fistula Fístula pilonidal OFFICIAL orcid:0009-0005-0193-5224, orcid:0000-0001-6276-958
en pt HP:0010770 rdfs:label Pilonidal fistula Fístula pilonidal OFFICIAL orcid:0009-0005-0193-5224|orcid:0000-0001-6276-958

Copy link
Contributor

Choose a reason for hiding this comment

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

No spaces, pipe-separated.

@dilzacampos
Copy link
Author

I hope it is correct now

Copy link
Contributor

@matentzn matentzn left a comment

Choose a reason for hiding this comment

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

Excellent, obrigado!

@matentzn matentzn merged commit 823c266 into obophenotype:main Feb 6, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants