Summary: | Curbside pickups - Show link if not configured | ||
---|---|---|---|
Product: | Koha | Reporter: | Tomás Cohen Arazi (tcohen) <tomascohen> |
Component: | Circulation | Assignee: | Tomás Cohen Arazi (tcohen) <tomascohen> |
Status: | Signed Off --- | QA Contact: | Testopia <testopia> |
Severity: | enhancement | ||
Priority: | P5 - low | CC: | david, gmcharlt, kyle.m.hall, oleonard, tomascohen |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
GIT URL: | Change sponsored?: | --- | |
Patch complexity: | String patch | Documentation contact: | |
Documentation submission: | Text to go in the release notes: | ||
Version(s) released in: | Circulation function: | Curbside pickups | |
Bug Depends on: | 40206 | ||
Bug Blocks: | |||
Attachments: |
Bug 40167: Curbside pickups - Show link if not configured
Bug 40167: Curbside pickups - Show link if not configured Bug 40167: (follow-up) Tweak message text |
Description
Tomás Cohen Arazi (tcohen)
2025-06-18 19:25:47 UTC
Created attachment 183397 [details] [review] Bug 40167: Curbside pickups - Show link if not configured If curbside pickups are not configured for the current branch, a correct message is displayed. It would be helpful to provide a link to the configuration page right there, if the patron has permissions to set it. This patch just adds that. To test: 1. Enable Curbside pickups 2. Make sure you are in a branch that doesn't have any rules for curbside pickups 3. Go to Circulation > Curbside pickups => SUCCESS: A message about curbside pickups not being configured is displayed 4. Apply this patch 5. Repeat 3 (reload?) => SUCCESS: The message is displayed, but you can now follow a link to configure the rules. 6. Sign off :-D Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Created attachment 183400 [details] [review] Bug 40167: Curbside pickups - Show link if not configured If curbside pickups are not configured for the current branch, a correct message is displayed. It would be helpful to provide a link to the configuration page right there, if the patron has permissions to set it. This patch just adds that. To test: 1. Enable Curbside pickups 2. Make sure you are in a branch that doesn't have any rules for curbside pickups 3. Go to Circulation > Curbside pickups => SUCCESS: A message about curbside pickups not being configured is displayed 4. Apply this patch 5. Repeat 3 (reload?) => SUCCESS: The message is displayed, but you can now follow a link to configure the rules. 6. Sign off :-D Signed-off-by: Tomas Cohen Arazi <tomascohen@theke.io> Signed-off-by: Owen Leonard <oleonard@myacpl.org> Created attachment 183401 [details] [review] Bug 40167: (follow-up) Tweak message text I've reworded the message a little bit in order to make it easier to translate and to make sure the link is descriptive. |