human-essentials icon indicating copy to clipboard operation
human-essentials copied to clipboard

[4987] Annual Reports range Export

Open douten opened this issue 5 months ago • 5 comments

Resolves #4987

Reviewer Notes

  • please test with various data, I've only used the seed data locally
  • if commits need to be squashed, lmk. I believe I need to push w/ force to do that, so I'm holding if off for now.

Description

  • new path to get bulk annual reports
    • /reports/annual_reports/range/{year_start}/{year_end}.csv ex: ../reports/annual_reports/range/2023/2024.csv
  • years in url must be 4 digits or Routing Error will be raised
  • the latest year of earliest_reporting_year and year_start will be used
  • the earliest year of the current year and year_end will be used
  • will use existing AnnualReport. If latest data is needed, the individual year's annual report will have to be recalculated

Type of change

  • New feature (non-breaking change which adds functionality)
  • Documentation update

How Has This Been Tested?

  • tested on page
  • made a small rspec test

Screenshots

https://github.com/user-attachments/assets/25aea7c4-b778-46d4-beca-3bdf181cfe08

douten avatar Jul 13 '25 00:07 douten

Thanks for the info @cielf I'll check back when there's an update then.

douten avatar Jul 14 '25 20:07 douten

Thanks for the info @cielf I'll check back when there's an update then.

I'm sorry -- I thought that we'd included the information that yes -- we can recalculate the reports. They do, however, still end up having different columns.

cielf avatar Aug 12 '25 07:08 cielf

@douten Are you still able to work on this? If not, we'll open it up for someone else.

cielf avatar Oct 30 '25 17:10 cielf

More than a month since we last heard from the developer -- I'm going to mark the issue as available (might take it myself).

cielf avatar Dec 11 '25 19:12 cielf

Notes on what I see that still needs to be fixed after adding the recalculation -- 1/ The "Please wait..." doesn't go away -- It doesn't on the individual report exports either. In Production. So -- perhaps not an absolutely necessary fix to put this forward, but recalculating everything could take a bit of time in prod, so I'd like to see it addressed. [Not happening today.]

cielf avatar Dec 11 '25 20:12 cielf