Add explanatory text to patron self-registration

Bug #2028411 reported by Jeff Davis
30
This bug affects 6 people
Affects Status Importance Assigned to Milestone
Evergreen
Confirmed
Wishlist
Unassigned

Bug Description

It would be cool if there was an easy way to add explanatory text to the patron self-registration form -- for example, a "what this page is for" blurb at the top, or a personal information collection notice above the submit button -- without having to modify register.tt2 directly.

Changed in evergreen:
assignee: nobody → Jeff Davis (jdavis-sitka)
importance: Undecided → Wishlist
Revision history for this message
Diane Disbro (ddisbro) wrote :

Yes! "Must be 18-years-old" and "Enter DOB in format shown" would save staff and patron time. It would also be helpful for individualized texts to be available at the system rather than the consortium level.

Revision history for this message
Jessica Woolford (jwoolford) wrote :

+1 for this idea and customizable text per org

Changed in evergreen:
status: New → Confirmed
tags: added: opac
tags: added: usability ux-form-hints
Revision history for this message
Jeff Davis (jdavis-sitka) wrote (last edit ):

Working branch user/jeffdavis/lp2028411-self-registration-blurbs-2 has a potential implementation:

https://git.evergreen-ils.org/?p=working/Evergreen.git;a=shortlog;h=refs/heads/user/jeffdavis/lp2028411-self-registration-blurbs-2

This branch adds a couple of settings to config.tt2 allowing you to add text at the top of the form (ctx.register_intro), immediately above the submit button (ctx.register_submit_notice), or both. Examples are provided but both settings are disabled by default. If we like this approach, I'll add a release note and target to the next release.

Another option would be to use a library setting. I went with config.tt2 because it feels like the right way to control what's displayed in the OPAC, and because it avoids the risk of an injection attack via library settings.

tags: added: needsreleasenote pullrequest
Changed in evergreen:
assignee: Jeff Davis (jdavis-sitka) → nobody
Revision history for this message
Jeff Davis (jdavis-sitka) wrote :

Release notes have been added to the branch.

Changed in evergreen:
milestone: none → 3.12-beta
tags: removed: needsreleasenote
Revision history for this message
Jane Sandberg (sandbergja) wrote :

I like it! Here's a signoff branch in the working repo: user/sandbergja/lp2028411-signoff -- thanks, Jeff!

tags: added: signedoff
Revision history for this message
Bill Erickson (berick) wrote :
Changed in evergreen:
assignee: nobody → Bill Erickson (berick)
tags: removed: signedoff
Changed in evergreen:
assignee: Bill Erickson (berick) → nobody
Revision history for this message
Jeff Davis (jdavis-sitka) wrote :

I included HTML markup in the example settings because the messages will often need to include markup in the real world -- my use case is a multi-paragraph message containing a link to an external privacy policy. I don't want to bikeshed though, I'm good with whatever gets this committed.

Revision history for this message
Jane Sandberg (sandbergja) wrote :

What if we split the difference, including both some HTML and some i18n in the example text, e.g.:

ctx.register_intro = "<p>" _ l("Please fill out the form below.") _ "</p>";

tags: added: needswork
removed: pullrequest
Changed in evergreen:
milestone: 3.12-beta → none
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.