-
Notifications
You must be signed in to change notification settings - Fork 34
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
PrivacyAgreement refactor #53
Comments
I like the ideas; let's also get you and Dave working on this together as a way to integrate him into the team. |
Great idea! |
I thought I saw some chatter between @annekainicUSDS and @dmethvin-gov about this -- can we make sure that we're chattering in the issues please, for transparency? Also, if I'm completely wrong and I dreamed that, my bad! |
I can see a strong case for having an independent page for the privacy agreement, so that it can be linked to and read outside of the form. Otherwise there's no way to reference it unless you're actually filling out the form, and there may be other constraints (such as user login) on getting to the form. As I understand it, the privacy agreement isn't really special. It's just a "mandatory" checkbox with label wording like "I have read and understand the privacy agreement" that includes a link to the privacy agreement text. That link can open a new window so it won't leave the form. As with all mandatory checkboxes, the form can't be submitted until it is checked, and the error message wording can emphasize that. If possible we should just use generic capabilities rather than creating a special case in the code. If we really want to enforce the need for a privacy agreement at the library level, we can look for a specifically named field, e.g. |
Things a user might want to customize:
|
Right now, the way the privacy agreement is included in the library is through both a separate component,
PrivacyAgreement
, that resides within the platform code on Vets.gov, which then redirects the user to a new page with the privacy agreement language.In order for this to be made generally usable by the library, we probably want to do a few things:
The text was updated successfully, but these errors were encountered: