Skip to content

Comments

Internationalization - Facet Category/Facet Label#5618

Closed
JayanthyChengan wants to merge 15 commits intoIQSS:developfrom
scholarsportal:dataverse-facetDisplay
Closed

Internationalization - Facet Category/Facet Label#5618
JayanthyChengan wants to merge 15 commits intoIQSS:developfrom
scholarsportal:dataverse-facetDisplay

Conversation

@JayanthyChengan
Copy link
Contributor

For both facet categories and Facet labels, key/value pairs are added to bundle files based on what solr returns and fetched into UI ( search-include-fragment.xhtml )

Related Issues

@coveralls
Copy link

Coverage Status

Coverage remained the same at 17.269% when pulling 2c9ccb5 on scholarsportal:dataverse-facetDisplay into d78b08d on IQSS:develop.

Copy link
Member

@pdurbin pdurbin left a comment

Choose a reason for hiding this comment

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

I left some comments and questions inline.

@@ -8,8 +8,8 @@ datasets=Ensembles de donn
newDataset=Nouvel ensemble de donn�es
Copy link
Member

Choose a reason for hiding this comment

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

When will this entire Bundle_fr.properties file be removed? It's the plan for installations who want French support to get the Bundle_fr.properties file from https://github.com/GlobalDataverseCommunityConsortium/dataverse-language-packs ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@pdurbin - Yes, I would like to create a separate issue to remove all french language files and Bundle_zh_CN.properties from codebase. Also to move all the English property files into a specific folder. Eg: under src/main/java/english/*.properties , so this folder can be synchronized automatically (through Jenkins setup) to the English folder in dataverse-language-packs repository. Later, translators can work for other languages. Please add your comments. Thanks


#astrophysics
Image=Image
Mosaic=Mosaic
Copy link
Member

Choose a reason for hiding this comment

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

Do we really want strings that were formerly in astrophysics.tsv (for example) in the main Bundle.properties file? Should they go in src/main/java/astrophysics.properties instead? Also, does metadatacustomization.rst need to be updated to mention facets and where to put the strings when creating a custom metadata block?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@pdurbin - The key added to the bundle.properties are the values returned from solr for facets. Is there any way to identify which metadata property file it corresponds to?

@scolapasta
Copy link
Contributor

closing in favor of #5697

@scolapasta scolapasta closed this Apr 24, 2019
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.

5 participants