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

Add missing codelist entry for collections #4

Merged
merged 1 commit into from Feb 14, 2017
Merged

Conversation

K-Phoen
Copy link

@K-Phoen K-Phoen commented Feb 14, 2017

Dans la même veine que la PR #3, j'ajoute un codelist manquant dans les types de collections.

Copy link

@cGuille cGuille left a comment

Choose a reason for hiding this comment

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

Faut pas mettre les :issue_number ?

@pmartin
Copy link

pmartin commented Feb 14, 2017

Ca vaudrait pas le coup de tout mettre à jour d'un coup, en fait ?
y'a l'air d'y avoir une task update_code_lists dans le binaire, qui ferait peut-être justement ça ?

@K-Phoen
Copy link
Author

K-Phoen commented Feb 14, 2017

J'arrive pas à faire tourner la task en local sur les fichiers que j'ai, et le lien vers editeur.com que je trouve dans le code m'envoie sur une page en 503 ¯\_(ツ)_/¯

@K-Phoen
Copy link
Author

K-Phoen commented Feb 14, 2017

Faut pas mettre les :issue_number ?

Ça marche sans, et je sais pas à quoi correspond ce truc :/

@pmartin
Copy link

pmartin commented Feb 14, 2017

aergh la task qui marche pas :-D

"issue number", je me dirais que ça correspond à la version des rulesets dans laquelle ça a été ajoutée
par exemple : http://www.editeur.org/files/ONIX%20for%20books%20-%20code%20lists/ONIX_BookProduct_Codelists_Issue_34.html == version 34

ah ben en 503 ici aussi, alors que ça marche "d'habitude" -- leur site doit être down, aujourd'hui ^^

@K-Phoen K-Phoen merged commit 39f6fe0 into master Feb 14, 2017
@K-Phoen K-Phoen deleted the collection-type branch February 14, 2017 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants