TECH - deplacement du calcul du score etabilssement dans un CRON

[JeromeBu]

No description provided.

[github-actions]

Review app:
https://if-dev-front-pr2239.osc-fr1.scalingo.io

[celineung]

answered_discussions peut être null. Avec postgres, la division null/total_discussions sera null, et une addition avec un null sera null aussi.

je propose:

Suggested change

	THEN (answered_discussions::float / total_discussions) * ${discussionCountCoefficient}
	THEN (COALESCE(answered_discussions::float, 0) / total_discussions) * ${discussionCountCoefficient}

ou sinon de modifier plus haut dans le WITH de discussion_counts:

sql`COUNT(DISTINCT 
  CASE WHEN exchanges.sender = 'establishment'::exchange_role THEN d.id
  ELSE 0 // par défaut sans ce else, ce sera NULL
  END)`.as(
            "answered_discussions",
          ),

[JeromeBu]

Il y a bien un coalesce mais à la sortie du select (line 171). Pour moi ça revient au même.

J'ai joué la query sur le replicat, et j'ai pas eu de problème (sachant que score est obligatoire, on aurait vu un problème en essayant de le mettre à null)