Closed ejbleys closed 5 years ago
@ejbleys thank you for pushing the fix! While you're at it, could we also change the "http://schemas.isotc211.org" to "https://schemas.isotc211.org"? The redirection happens anyway, but some XML schema parsers can't handle the HTTP->HTTPS redirect.
OK Do you want me to go through all XSDs and XMLs to move to https://schemas.isotc211.org? https://schemas.isotc211.org/? Logically other people’s XMLs would fail as there would be a mismatch on nameSpace locations. Advice sought: a) Do it all? b) All in new version? c) Just XMLs?
Cheers e Evert Bleys 4 Tudor Place HUGHES ACT 2605 Australia +61 (0)2 62811773 +61 (0)411 483 876 ejbleys@gmail.com Skype ejbleijs@gmail.com
On 2019-09-09, at 5:45 pm, Ronald Tse notifications@github.com wrote:
@ejbleys https://github.com/ejbleys thank you for pushing the fix! While you're at it, could we also change the "http://schemas.isotc211.org http://schemas.isotc211.org/" to "https://schemas.isotc211.org https://schemas.isotc211.org/"? The redirection happens anyway, but some XML schema parsers can't handle the HTTP->HTTPS redirect.
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/ISO-TC211/schemas/pull/6?email_source=notifications&email_token=AIBGJZ46CWI5EJWAQT2Z3LTQIX5IBA5CNFSM4IUXW2RKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6GRQCQ#issuecomment-529340426, or mute the thread https://github.com/notifications/unsubscribe-auth/AIBGJZ2STYM3BTKMSAHDF5LQIX5IBANCNFSM4IUXW2RA.
@ejbleys that'd be awesome! IMHO (a) is best:
I'm not sure whether the http://schemas.isotc211.org/...
URIs were used much prior to the move to HTTPS -- they were down for a long time prior to migration. If they weren't used by many the big-bang approach will save us effort and provides consistency down the road.
@ejbleys could we merge this? Thanks!
Please merge Evert Bleys 4 Tudor Place HUGHES ACT 2605 Australia +61 (0)2 62811773 +61 (0)411 483 876 ejbleys@gmail.com Skype ejbleijs@gmail.com
On 2019-11-15, at 3:58 am, Ronald Tse notifications@github.com wrote:
@ejbleys https://github.com/ejbleys could we merge this? Thanks!
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/ISO-TC211/schemas/pull/6?email_source=notifications&email_token=AIBGJZZXPQAL6PUZCK6RUSTQTV7TDA5CNFSM4IUXW2RKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEECRCJQ#issuecomment-553980198, or unsubscribe https://github.com/notifications/unsubscribe-auth/AIBGJZ6XCLH5ATCQ7PCC4PLQTV7TDANCNFSM4IUXW2RA.
Complaint received that code list entries in mir:codelists.xml "intelligenceMilitary" and "boundaries" both had trailing space in code value (gco:ScopeName) and Code list name (cat:CT_CodeListValue) Spaces have been removed - no other changes made.