Review Notifications 'Next Send' is stored in config yet gets changed automatically

Also combined patch attached for #3372256 + #3372260 since these have a merge conflict in the update hook (since both require a new update hoo)

@scott_euser thank you for this work. I will do my best to review this today or tomorrow so as to get the module updated quickly.

I've reviewed the patch and it mostly looks good. There are two things that I need to address:

- there is call to the config service in the hook_cron function that can be removed since the date is going to now be stored in state.
- there needs to be accounting for the case where there is no saved state.

Hi @aaronpinero,

Thanks for the quick review on both of these!

For no state saved, what do you think of:

1. Set state to current time +1 day
2. Update field description on config page to say that if not set, state will be set to current time +1 day
3. Add a note to README to say that if not set, state will be set to current time +1 day

@scott_euser yes, that seems fine.

Thanks, I have updated the MR to reflect the above. Ready for review again.

Automatically closed - issue fixed for 2 weeks with no activity.