Skip to content

Use the ISOYEAR instead of the YEAR when grouping by week

Yannis Roussos requested to merge 7-use-isoyear-in-the-per-week-models into master

We this MR we:

  • Add a macro for extracting the ads insights reports date and their date parts, including the iso year
  • Update all base models to use that macro (and also extract the iso year for each report)
  • Update all XXX_per_week models to use the insights_iso_year instead of the insights_year

Also some grouping reorganizations to improve our consistency and make our 3rd grade teachers happy.

Closes #7 (closed)

Merge request reports