Katie Smith
Katie Smith
@spatel033 Yes, it does need to be removed from the database but that will be done in a separate PR after this (along with removing all the [other unused service...
@joybytes @spatel033 I've removed the last commit from this - the one which removed the unused constants. That part will need to be deployed later as part of tidying the...
Thanks @quis branching off a branch was giving errors in Concourse, so I've now changed the base branch to main and will just need to make sure this gets rebased...
Now updated just to change `from` to `From` on the signed out page as agreed
Looking at the existing code comment, it seems like we intentionally didn't order the notifications when getting 50,000 to delete. It doesn't say _why_, but I was wondering if there...
Is it ok for the delivery times to be displayed now, or does this need to wait until the feature is live?
You've probably already seen this but not all the headings are consistent, so that might need to be looked at before this could be merged. No major differences, just things...