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

Fix issue where serializing vocabulary terms into '|' syntax failed when values contained non-ascii characters #61

Merged
merged 4 commits into from
Aug 14, 2018

Conversation

datakurre
Copy link
Member

No description provided.

@datakurre datakurre force-pushed the datakurre-vocabularies branch from 0a513f5 to d630820 Compare August 13, 2018 11:53
@datakurre datakurre requested a review from tomgross August 13, 2018 11:57
@datakurre datakurre changed the title [WIP] Add more tests for non-ascii vocabulary values Fix issue where serializing vocabulary terms into '|' syntax failed when values contained non-ascii characters Aug 13, 2018
@jensens jensens merged commit f50e596 into master Aug 14, 2018
@jensens jensens deleted the datakurre-vocabularies branch August 14, 2018 07:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants