-
Notifications
You must be signed in to change notification settings - Fork 30
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
Change xs:string to internationalizedText as appropriate #77
Comments
I started working on this in branch Also, here are the elements to update from the issue #39 comments: ballot_response.txt:
candidate.txt:
contest.txt:
custom_ballot.txt:
early_vote_site.txt: (doesn't exist?)
election.txt:
election_official.txt:
polling_location.txt:
referendum.txt: (became BallotMeasureContest)
source.txt:
election.txt:
party.txt:
|
@demcg Yes, I included Candidate.BallotName for that reason. Thanks. I'm in the process of typing up some thoughts about this issue and others. |
Okay, I created PR #92 to address this. I think the PR takes care of a lot of the easy and/or obvious stuff, but I think it would be good to start thinking about a more careful approach and rules around this going forward. Here are some thoughts to consider around this, to give you an idea:
|
…ationalized-text Address issue #77: change xs:string to InternationalizedText
Closed by PR #92. |
@cjerdonek All excellent points.
True, but I think this is a solid start and probably needs some additional input from other states/localities on whether this is something that should be consolidated under a "display" field or should be "built" by the end user.
Anecdotally, I know there are a lot of non-latin languages that, while there is a separate numeric system, using Arabic numbers is accepted. We'll still need to gather more data on the needs of local communities in more places.
Again, I think we'll need broader feedback here.
Interesting. This probably does need it's own display field, but I think what we have is solid for now. At some point, I imagine we'll get into a very interesting "unwritten languages" discussion :) |
Thanks, @jungshadow. And yes, to be clear, my comment was meant as food for thought for the future as opposed to a suggestion to do something more about it now. |
This issue is to change "xs:string" to "internationalizedText" as appropriate.
This is something that I believe was done in one of the earlier PR's, but then got lost. I can take care of this by reviewing the old comments and PR's, and then create a new PR. I will wait until after the style stuff is done before making the PR.
The text was updated successfully, but these errors were encountered: