Skip to content

Fix progress-report org summary error - #7662

Open
FestiveKyle wants to merge 6 commits into
masterfrom
fix/progress-report-summaries
Open

FestiveKyle wants to merge 6 commits into
masterfrom
fix/progress-report-summaries

Conversation

@FestiveKyle

Copy link
Copy Markdown
Collaborator

This PR fixes a bug in the progress-report during reading of the latest summary. It also has the following changes:

  • Skip orgs without summaries
  • Add affiliation permission gate to ensure pending users don't get emails
  • Fix logger service name
  • Add NOTIFICATION_ORG_PROGRESS_REPORT to .env.example and cloudbuild
  • Remove mocked logger
  • Improve tests (real DB and Notify test API key instead of mocks)

@FestiveKyle
FestiveKyle marked this pull request as ready for review October 15, 2026 17:16

This branch has not been deployed

No deployments
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.

1 participant