Add a consent requirement for policies (privacy and terms of service).

Review Request #9959 — Created May 17, 2018 and submitted

Information

Djblets
release-1.0.x
272b5a5...

Reviewers

This change adds a common consent requirement for policy enforcement. In
the case where an admin has specified their privacy policy and/or terms
of service, they likely are running a public service where they'll need
their users to accept them. This consent requirement isn't actually used
inside of Djblets, but can be added to a registry in a Djblets-using
application and checked therein.

Used in conjunction with a change in Review Board.

Description From Last Updated

Typo in the first line of the description: "IN"

chipx86chipx86

I kind of don't like saying "your server administrator" for the case of RBCommons. Can we have this take an …

chipx86chipx86

As Barret noticed during the Splat work, we need to mark safe in a lazy way. Right now, we're lazy-localizing …

chipx86chipx86
chipx86
  1. 
      
  2. Show all issues

    Typo in the first line of the description: "IN"

  3. djblets/privacy/consent/common.py (Diff revision 1)
     
     
     
     
    Show all issues

    I kind of don't like saying "your server administrator" for the case of RBCommons. Can we have this take an optional contact text as well? Default that to "contact your administrator?" Then for RBCommons, we can have it say "contact support."

  4. djblets/privacy/consent/common.py (Diff revision 1)
     
     
     
     
     
     
    Show all issues

    As Barret noticed during the Splat work, we need to mark safe in a lazy way. Right now, we're lazy-localizing and then marking safe, making it choose a locale, but since this is global it's going to be the wrong one for most languages. This will need to use the function he's adding to Djblets instead.

    1. Also, plugging in values will immediately translate the string. We might need to just make these properties.

    2. Yeah, that's where I was headed.

  5. 
      
david
chipx86
  1. Ship It!
  2. 
      
david
Review request changed
Status:
Completed
Change Summary:
Pushed to release-1.0.x (8529497)