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: ajout d'une table iMER (intentions de mise en relation) #223

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

Conversation

laurinehu
Copy link
Contributor

🥝 Besoin

  • suivi des iMER sur les services d·i dans le cadre de l'XP score de qualité
  • mesure de la rétention avec un focus sur l'iMER

🍅 Solution

Création d'une table contenant les intentions de mise en relation.

@laurinehu laurinehu self-assigned this Feb 5, 2025
@laurinehu laurinehu requested review from jbuget and ggounot February 5, 2025 19:06
Copy link
Contributor

@jbuget jbuget left a comment

Choose a reason for hiding this comment

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

Quelques détails de nommage

analytics/models/intermediate/int_iMER.sql Outdated Show resolved Hide resolved
analytics/models/intermediate/int_iMER.sql Outdated Show resolved Hide resolved
analytics/models/intermediate/int_iMER.sql Outdated Show resolved Hide resolved
analytics/models/intermediate/int_iMER.sql Outdated Show resolved Hide resolved
analytics/models/intermediate/int_iMER.sql Show resolved Hide resolved
@laurinehu laurinehu requested a review from jbuget February 7, 2025 09:58
jbuget
jbuget previously approved these changes Feb 7, 2025
Copy link
Contributor

@jbuget jbuget left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@jbuget jbuget enabled auto-merge (squash) February 7, 2025 15:02
@jbuget jbuget force-pushed the feat/metabase-imer-tables branch from 79fc7a8 to 6541d16 Compare February 11, 2025 08:13
@service-dev-gip-inclusion

Ajout d'une table iMER

@ggounot
Copy link
Contributor

ggounot commented Feb 11, 2025

Super la documentation web ! Ce serait bien de la mettre à disposition constante sur un serveur.
image

Copy link
Contributor

@ggounot ggounot left a comment

Choose a reason for hiding this comment

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

J'obtiens cette erreur en exécutant dbt run :

ERROR creating sql table model public_intermediate.int_iMER ........... [ERROR in 0.07s]
Database Error in model int_iMER (models/intermediate/int_iMER.sql)
  each UNION query must have the same number of columns
  LINE 26:     'mobilisation'            as kind,
               ^
  compiled code at target/run/dora/models/intermediate/int_iMER.sql

@ggounot
Copy link
Contributor

ggounot commented Feb 11, 2025

Je ne comprends pas bien le sens qu'il y a à combiner orientations et mobilisations. Une orientation aura forcément généré une mobilisation, non ?

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.

4 participants