combine two tables in TPAC patron notification preferences page
Bug #1613381 reported by
Galen Charlton
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Evergreen |
Fix Released
|
Wishlist
|
Unassigned |
Bug Description
The TPAC My Account notification preferences page currently defines two tables:
- various hardcoded options for hold and SMS notifications
- preferences coming from opt-in settings for Action/Trigger event definitions
The two tables have slightly different styles, which can look odd, but they also don't express a distinction that patrons necessarily care about: i.e., I as a patron may want to tweak which notifications I get from the library, but I don't particularly care whether that preference arises from an opt-in setting or from a static YAOUS.
I propose to merge the two tables into a single table.
Evergreen master
Changed in evergreen: | |
milestone: | none → 2.11-beta |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
A patch is available in the user/gmcharlt/ lp1613381_ combine_ patron_ notification_ prefs_tables branch of the working repository:
http:// git.evergreen- ils.org/ ?p=working/ Evergreen. git;a=shortlog; h=refs/ heads/user/ gmcharlt/ lp1613381_ combine_ patron_ notification_ prefs_tables