-
Notifications
You must be signed in to change notification settings - Fork 28
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
Don't break identifiers #37
Comments
@Floppy absolutely - we need to redirect / alias and have a proper deprecation process. |
Proposed a JSON representation for the deprecated IDs in #38 |
Great. Just as a BTW you may be interested in the proposal in okfn/opendefinition#7 (note we will promise no breakage - e.g. redirects will be put in ...) |
Will have to move off gh-pages to implement redirects usefu to machine clients. |
That may be irrelevant though, depending on how you're using gh-pages. |
That just generates HTML with javascript redirects. You can see it in action at http://opendefinition.org/okd/ |
yeah, sorry, I just read through properly and realised that myself. Ignore me :) |
https://github.com/okfn/licenses/pull/69/files#diff-df8c50fbd9842b77c087e6defc698cfa has a proposed rename and adds a |
@mlinksva i think that could work. @Stephen-Gates any thoughts at your end. |
@rufuspollock I've highlighted this thread to ODI Labs who I believe use the data. |
It'd appear its consumed in https://github.com/theodi/odlifier/blob/60d8d808f3b283a7c58d8c727ef0705ae30962f9/lib/odlifier.rb which is used in 3 other ODI projects https://github.com/theodi/odlifier/network/dependents Hopefully it'll be easy for them to become |
I'm not there any more, but yes, one of us could easily update |
@Floppy I made a crude attempt at theodi/odlifier#56 ... untested, I'm not good at Ruby and don't grok the VCR stuff in tests. 😬 🗑 Anyway, I'll merge #69 (changes |
Has been some months, assume nobody cares so merged #69 🎉 |
We rely on the license IDs that you define here for other apps. When they change (like for capitalisation, or renaming), it breaks things downstream that depend on them.
I accept renaming needs to happen, but when it does, perhaps older deprecated IDs could be kept around with some sort of link to the new ID in its JSON representation?
The text was updated successfully, but these errors were encountered: