Bugfix/Process single batch notifications

HalcyonJAC commented 3 months ago

What's included?

Issue:

Fix:

Rename cloud function from processNotifications to processNotificationsNow.

Who should test?

✅ Product owner ✅ Developer

How to test?

Go to "Notifications" section and hit "Process now (single batch)" button.
Check if the button works correctly.

Risk - how likely is this to impact other areas?

🟢 No risk - this is a self-contained piece of work

Additional context

Demo:

https://github.com/jac-uk/admin/assets/79906532/2a368574-b043-4259-9c9a-5d683c5a5ee6

Related permissions

Have permissions been considered for this functionality?

No permission changes required
Permissions have been added / updated. Details:

PREVIEW:DEVELOP can be OFF, DEVELOP or STAGING

github-actions[bot] commented 3 months ago

Visit the preview URL for this PR (updated for commit 97f71ec):

https://jac-admin-develop--pr2474-bugfix-process-singl-1mmq639d.web.app

_{(expires Wed, 31 Jul 2024 09:39:03 GMT)}

_{🔥 via Firebase Hosting GitHub Action 🌎}

_{Sign: 4e92cf51659207b0ae3509dc5c40edde50edfec0}

nickaddy commented 3 months ago

@HalcyonJAC After clicking Process now and the function having run, I would expect the list on screen to clear and items move to the Sent tab, but the items remain on screen. Is this correct?

jac-uk / admin