Right now, it displays a warning message when you navigate to Circulation > Curbside pickups, but don't have them configured for the current branch.
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.