Date attributes can now be repeated, so we should update the hint on the administration page: If checked, attribute will be a date. Date attributes cannot be repeatable or linked to an authorised value category.
Created attachment 167319 [details] [review] Bug 36922: Patron attributes - update hint text for "Is a date" The date field for patron attributes is now repeatable (enhancement made in Koha 24.05 with bug 32610). This updates the hint text to reflect the change. Test plan: 1. Go to add a patron attribute (Administration > Patrons and circulation > Patron attribute types > + New patron attribute type ) 2. Note that the hint for the "Is a date" field says "...Date attributes cannot be repeatable or linked to an authorised value category.". 3. Apply the patch. 4. Reload the page. 5. Note that the hint now says "...Date attributes are repeatable, but cannot be linked to an authorised value category." 6. Sign off D: Signed-off-by: David Nind <david@davidnind.com>
Created attachment 167337 [details] [review] Bug 36922: Patron attributes - update hint text for "Is a date" The date field for patron attributes is now repeatable (enhancement made in Koha 24.05 with bug 32610). This updates the hint text to reflect the change. Test plan: 1. Go to add a patron attribute (Administration > Patrons and circulation > Patron attribute types > + New patron attribute type ) 2. Note that the hint for the "Is a date" field says "...Date attributes cannot be repeatable or linked to an authorised value category.". 3. Apply the patch. 4. Reload the page. 5. Note that the hint now says "...Date attributes are repeatable, but cannot be linked to an authorised value category." 6. Sign off D: PA amended: 'are repeatable' -> 'can be repeatable' Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Pedro Amorim <pedro.amorim@ptfs-europe.com>
@David I changed the text slightly: 'are repeatable' -> 'can be repeatable' Date fields are optionally repeatable, so it may or may not be repeatable. Do you agree with this? Thanks.
(In reply to Pedro Amorim from comment #3) > @David I changed the text slightly: > 'are repeatable' -> 'can be repeatable' > > Date fields are optionally repeatable, so it may or may not be repeatable. > Do you agree with this? Thanks. Thanks Pedro. I generally try to write things in the active voice, but I'm OK with the change. David
Created attachment 167400 [details] [review] Bug 36922: Patron attributes - update hint text for "Is a date" The date field for patron attributes is now repeatable (enhancement made in Koha 24.05 with bug 32610). This updates the hint text to reflect the change. Test plan: 1. Go to add a patron attribute (Administration > Patrons and circulation > Patron attribute types > + New patron attribute type ) 2. Note that the hint for the "Is a date" field says "...Date attributes cannot be repeatable or linked to an authorised value category.". 3. Apply the patch. 4. Reload the page. 5. Note that the hint now says "...Date attributes are repeatable, but cannot be linked to an authorised value category." 6. Sign off D: PA amended: 'are repeatable' -> 'can be repeatable' Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Pedro Amorim <pedro.amorim@ptfs-europe.com> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
Thanks for all the hard work! Pushed to main for the next 24.11.00 release as RM Assistant
Backported to 24.05.x for upcoming 24.05.02
Depends on Bug 32610 not in 23.11.x
No change required to the manual - screenshot is up-to-date (added by manual changes for bug 32610), no changes to the text required.