Skip to content

Hotfix so uses correct end day and split switch into readable if statemet#1146

Closed
Paul Cooper (PaulCooperWorkJustice) wants to merge 1 commit into
developfrom
CFODEV-1168-HotFix-Recent-Approved-Activities
Closed

Hotfix so uses correct end day and split switch into readable if statemet#1146
Paul Cooper (PaulCooperWorkJustice) wants to merge 1 commit into
developfrom
CFODEV-1168-HotFix-Recent-Approved-Activities

Conversation

@PaulCooperWorkJustice

Copy link
Copy Markdown
Contributor

🔗 Related Work

  • Closes: #
  • Related: #

📌 Summary

What does this PR do? Keep it short but clear.


🎯 Purpose / Motivation

Why does this change exist? What problem are we solving?


🧠 Approach

Key implementation details, trade-offs, or design decisions.
Mention anything reviewers should pay attention to.


🔄 Changes

  • Added:
  • Updated:
  • Removed:

🧪 How to Test

Step-by-step instructions to verify this works.

Expected result:

What should happen if everything is correct?


📸 Screenshots / Output (if applicable)

UI changes, logs, API responses, etc.


⚠️ Risks & Impact

  • Breaking change
  • Database change
  • Performance impact
  • Security impact

Details:

Anything reviewers should be cautious about.


🙋 Notes for Reviewers

Anything specific you want feedback on.
e.g. "Unsure about approach in X", "Focus on Y logic"

…ement.

Signed-off-by: Paul Cooper <Paul.Cooper5@Justice.gov.uk>
@PaulCooperWorkJustice Paul Cooper (PaulCooperWorkJustice) added Needs Review This pull request is awaiting review Bug Fix labels Jun 18, 2026
@PaulCooperWorkJustice Paul Cooper (PaulCooperWorkJustice) deleted the CFODEV-1168-HotFix-Recent-Approved-Activities branch June 18, 2026 10:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Bug Fix Needs Review This pull request is awaiting review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant