Skip to content

Ad hoc reports#4884

Merged
frjo merged 19 commits into
mainfrom
feature/ad-hoc-reports
Jun 24, 2026
Merged

Ad hoc reports#4884
frjo merged 19 commits into
mainfrom
feature/ad-hoc-reports

Conversation

@frjo

@frjo frjo commented Jun 20, 2026

Copy link
Copy Markdown
Member

Fixes #4529

Adds ad-hoc report management for projects, giving staff the ability to manually add/edit/delete report due dates rather than relying solely on automated scheduling.

Test Steps

  • Confirm that added due dates work as expected.
  • Confirm that only staff can delete and edit due dates.
  • Confirm that notification are created when staff delete a report.

@frjo frjo added Type: Feature This is something new (not an enhancement of an existing thing). Type: Minor Minor change, used in release drafter Type: Enhancement This is an improvement of an existing thing (not a new thing, which would be a feature). and removed Type: Feature This is something new (not an enhancement of an existing thing). Type: Enhancement This is an improvement of an existing thing (not a new thing, which would be a feature). labels Jun 20, 2026
@frjo frjo force-pushed the feature/ad-hoc-reports branch from 449058e to 91242d2 Compare June 21, 2026 12:52
@wes-otf wes-otf self-requested a review June 22, 2026 17:05

@wes-otf wes-otf left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This looks and works great! I just had a super small nitpick around due vs end date language

Comment thread hypha/apply/projects/reports/forms.py Outdated

class ReportAddDateForm(forms.Form):
end_date = forms.DateField(
label=_("Report end date"),

@wes-otf wes-otf Jun 23, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Might make more sense for this to be "due date" as it'd be in line with the edit prompt

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Good catch.

@wes-otf

wes-otf commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

Also maybe adding a description to the PR just for posterity when we do merge it in

@frjo frjo force-pushed the feature/ad-hoc-reports branch from 91242d2 to dd5a09c Compare June 24, 2026 09:33
@frjo frjo requested a review from wes-otf June 24, 2026 09:37
@frjo frjo merged commit 7ca385d into main Jun 24, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type: Feature This is something new (not an enhancement of an existing thing). Type: Minor Minor change, used in release drafter

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Ad-hoc reporting

3 participants