Bug 6978 - Notify the user on his account page when it is almost expired.
Summary: Notify the user on his account page when it is almost expired.
Status: CLOSED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: OPAC (show other bugs)
Version: 3.6
Hardware: All All
: PATCH-Sent (DO NOT USE) enhancement (vote)
Assignee: Frédérick Capovilla
QA Contact: Bugs List
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-10-05 19:36 UTC by Frédérick Capovilla
Modified: 2013-12-05 19:59 UTC (History)
3 users (show)

See Also:
Change sponsored?: Sponsored
Patch complexity: ---
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:


Attachments
Notify the user on his account page when his account is almost expired (2.84 KB, patch)
2011-10-05 19:36 UTC, Frédérick Capovilla
Details | Diff | Splinter Review
Corrected patch (3.09 KB, patch)
2011-10-12 14:16 UTC, Frédérick Capovilla
Details | Diff | Splinter Review
Signed-off patch (3.40 KB, patch)
2011-10-12 14:42 UTC, Owen Leonard
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Frédérick Capovilla 2011-10-05 19:36:18 UTC Comment hidden (obsolete)
Comment 1 Frédérick Capovilla 2011-10-06 16:54:38 UTC
Comment on attachment 5722 [details] [review]
Notify the user on his account page when his account is almost expired

This patch was sent by mistake. Please wait before adding to 3.6 .
Comment 2 Frédérick Capovilla 2011-10-07 14:50:14 UTC
Comment on attachment 5722 [details] [review]
Notify the user on his account page when his account is almost expired

Small communication problem. We had the authorization from our client (CCSR) all along to release this code.
Comment 3 Owen Leonard 2011-10-11 16:09:11 UTC
This feature only works if your dateformat system preference is set to ISO.
Comment 4 Frédérick Capovilla 2011-10-12 14:16:25 UTC Comment hidden (obsolete)
Comment 5 Owen Leonard 2011-10-12 14:42:45 UTC
Created attachment 5856 [details] [review]
Signed-off patch

Works now with any dateformat system setting, and correctly tracks NotifyBorrowerDeparture setting. This is a very nice addition.
Comment 6 Ian Walls 2011-10-18 15:25:16 UTC
Nice and clean, adds a valuable feature based on existing system preference values.  This does introduce many new strings, however, so it may need to wait.  Passing QA, and leaving to RM's discretion for integration.
Comment 7 Chris Cormack 2011-10-20 01:01:28 UTC
Feed back sought from translators, worth including even though we are in string freeze?
Comment 8 Paul Poulain 2011-11-06 10:39:00 UTC
patch pushed, please test
Comment 9 Jared Camins-Esakov 2012-05-23 12:13:43 UTC
Included in 3.6 prior to 3.6.5.