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

feat(pipeline): add imilo as a new data source #365

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

YannickPassa
Copy link
Contributor

hlecuyer and others added 4 commits January 8, 2025 19:25
addition of the nullif trim

rename imilo services to match the source

staging

staging

fix staging columns

staging additions
int addition

int additions

int

fix : add the necessary imilo annotations for quality

+ little bug in the int_quality_stats because
  - the iteration is not technically over (there still is
    imilo_offres_structures)
  - but the next one is not a qualifying source (no kind) so
    the iteration fails
  - switch to loop.first and prepend UNION ALL to avoid any further issue

int additions
@YannickPassa YannickPassa requested a review from vperron January 8, 2025 18:29
@YannickPassa YannickPassa self-assigned this Jan 8, 2025
@YannickPassa YannickPassa requested a review from vmttn as a code owner January 8, 2025 18:29
@vperron vperron force-pushed the YannickPassa/feat/add_milo_source_to_the_api branch from 8350352 to 48297b3 Compare January 9, 2025 15:30
@YannickPassa YannickPassa force-pushed the YannickPassa/feat/add_milo_source_to_the_api branch from 48297b3 to 279e10d Compare January 9, 2025 15:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants