Skip to content
This repository has been archived by the owner on Jul 25, 2024. It is now read-only.

SMD-758: make Towns Fund Round 4 inactive #153

Merged
merged 3 commits into from
Apr 9, 2024
Merged

SMD-758: make Towns Fund Round 4 inactive #153

merged 3 commits into from
Apr 9, 2024

Conversation

gidsg
Copy link
Contributor

@gidsg gidsg commented Apr 8, 2024 •
edited
Loading

Change description

make Towns Fund Round 4 inactive (now all LAs have submitted)

  • Unit tests and other appropriate tests added or updated
  • README and other documentation has been updated / added (if needed)
  • Commit messages are meaningful and follow good commit message guidelines (e.g. "FS-XXXX: Add margin to nav items preventing overlapping of logo")

How to test

  • Towns Fund should not appear on dashboard (if user has Towns Fund role)
  • /upload/TF/4 should 401

Screenshots of UI changes (if applicable)

@gidsg gidsg marked this pull request as draft April 8, 2024 15:49
@gidsg gidsg force-pushed the smd-758-close-r4 branch from a274ed1 to 69d340b Compare April 8, 2024 16:30
@gidsg gidsg marked this pull request as ready for review April 8, 2024 16:42
Joel-Pearce
Joel-Pearce approved these changes Apr 9, 2024
View reviewed changes
Copy link
Contributor

@Joel-Pearce Joel-Pearce left a comment

Choose a reason for hiding this comment

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

LGTM

@gidsg gidsg merged commit e2dce21 into main Apr 9, 2024
2 checks passed
@gidsg gidsg deleted the smd-758-close-r4 branch April 9, 2024 11:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@Joel-Pearce Joel-Pearce Joel-Pearce approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@gidsg @Joel-Pearce