Bug 35279

Summary: Add fallback for WELCOME notice to allow 'print' when patrons are missing email address
Product: Koha Reporter: Martin Renvoize (ashimema) <martin.renvoize>
Component: NoticesAssignee: Martin Renvoize (ashimema) <martin.renvoize>
Status: Needs documenting --- QA Contact: Testopia <testopia>
Severity: enhancement    
Priority: P5 - low CC: david, fridolin.somers
Version: MainKeywords: Sandbox
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=34704
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=30287
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33478
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=33260
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
24.05.00
Circulation function:
Bug Depends on:    
Bug Blocks: 38087    
Attachments: Bug 35279: Add fallback to print for 'Send welcome email'
Bug 35279: Add fallback to print for 'Send welcome email'
Bug 35279: Add fallback to print for 'Send welcome email'
Bug 35279: Add fallback to print for 'Send welcome email'

Description Martin Renvoize (ashimema) 2023-11-07 14:06:32 UTC
When testing bug 34704 I realised it's not especially trivial to trigger a print notice to be generated.  I think it's a good idea to allow fallback to print for the WELCOME notice when calling 'Send welcome email' from the patrons profile.
Comment 1 Martin Renvoize (ashimema) 2023-11-07 14:12:12 UTC Comment hidden (obsolete)
Comment 2 Martin Renvoize (ashimema) 2023-11-07 14:33:52 UTC
Created attachment 158601 [details] [review]
Bug 35279: Add fallback to print for 'Send welcome email'

We update the 'Send welcome email' button to read 'Send welcome notice'
and add a fallback to enqueue a notice for print when the user doesn't
have an email address defined.

Test plan
1) Apply patch
2) Use the 'Henry Acevedo' user who doesn't yet have a valid email
   address.
3) Click the 'Send welcome notice' button available from the 'more'
   menu.
4) Note that no notice is queued at all.
5) Copy the content of the 'email' WELCOME notice into the 'print'
   template.
5) Repeat step 3.
6) Note that this time a 'print' type notice is queued for the user.
Comment 3 David Nind 2023-11-07 17:37:36 UTC
Created attachment 158612 [details] [review]
Bug 35279: Add fallback to print for 'Send welcome email'

We update the 'Send welcome email' button to read 'Send welcome notice'
and add a fallback to enqueue a notice for print when the user doesn't
have an email address defined.

Test plan
1) Apply patch
2) Use the 'Henry Acevedo' user who doesn't yet have a valid email
   address.
3) Click the 'Send welcome notice' button available from the 'more'
   menu.
4) Note that no notice is queued at all.
5) Copy the content of the 'email' WELCOME notice into the 'print'
   template.
5) Repeat step 3.
6) Note that this time a 'print' type notice is queued for the user.

Signed-off-by: David Nind <david@davidnind.com>
Comment 4 Kyle M Hall (khall) 2024-01-26 17:42:52 UTC
Created attachment 161547 [details] [review]
Bug 35279: Add fallback to print for 'Send welcome email'

We update the 'Send welcome email' button to read 'Send welcome notice'
and add a fallback to enqueue a notice for print when the user doesn't
have an email address defined.

Test plan
1) Apply patch
2) Use the 'Henry Acevedo' user who doesn't yet have a valid email
   address.
3) Click the 'Send welcome notice' button available from the 'more'
   menu.
4) Note that no notice is queued at all.
5) Copy the content of the 'email' WELCOME notice into the 'print'
   template.
5) Repeat step 3.
6) Note that this time a 'print' type notice is queued for the user.

Signed-off-by: David Nind <david@davidnind.com>
Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
Comment 5 Katrin Fischer 2024-03-22 14:48:15 UTC
Pushed for 24.05!

Well done everyone, thank you!
Comment 6 Fridolin Somers 2024-05-23 08:26:49 UTC
Not backported to 23.11.x