Skip to content
This repository has been archived by the owner on Nov 22, 2023. It is now read-only.

(PC-1587) Init import Titelive Things #489

Merged
merged 3 commits into from
Mar 18, 2019

Conversation

DanielBaudry
Copy link
Contributor

No description provided.

@DanielBaudry DanielBaudry requested a review from arnoo March 14, 2019 08:55
if self.infos['url_extrait_pdf'] != '':
thing.mediaUrls.append(self.infos['url_extrait_pdf'])

def get_extraData_from_infos(self):
Copy link
Contributor

Choose a reason for hiding this comment

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

Pour le coup ça aurait été bien de le factoriser quelque part, lui, non ? car c'est tout le meme que l'autre je suppose ?


return extraData

def get_infos_from_data_line(self, elts: []):
Copy link
Contributor

Choose a reason for hiding this comment

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

idem

Copy link
Contributor

Choose a reason for hiding this comment

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

genre un titelive_utils

Copy link
Contributor

Choose a reason for hiding this comment

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

meme si ce fichier en particulier ne va servir qu'une fois a priori, les gens ne le sauront pas forcément s'ils doivent faire des mises à jour

return data_lines


def get_thing_type_and_extra_data_from_titelive_type(titelive_type):
Copy link
Contributor

Choose a reason for hiding this comment

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

idem

@DanielBaudry DanielBaudry merged commit 1376655 into master Mar 18, 2019
@DanielBaudry DanielBaudry deleted the pc-1587-script-import-titelive branch March 18, 2019 15:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
2 participants