Project

General

Profile

Actions
Copy link

Story #2526

closed

Make user-acceptance of TOS optional

Added by Phil Hodgson almost 11 years ago. Updated over 10 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Phil Hodgson
Category:
-
Target version:
Testing and Upgrading + PGP.ca features
Start date:
04/01/2014
Due date:
% Done:

100%

Estimated time:
(Total: 0.00 h)
Story points:
-

Subtasks 2 (0 open — 2 closed)

Task #2537: Add config.yml entry for "ensure_tos" and hook it up to the authorization filterResolvedPhil Hodgson04/01/2014

Actions
Task #2538: Review 2526-optional-tosResolvedPhil Hodgson04/01/2014

Actions
Actions
Copy link
 #1

Updated by Ward Vandewege almost 11 years ago

  • Parent task deleted (#2501)
Actions
Copy link
 #2

Updated by Ward Vandewege almost 11 years ago

  • Tracker changed from Task to Story
Actions
Copy link
 #3

Updated by Anonymous almost 11 years ago

  • Status changed from New to Resolved
  • % Done changed from 50 to 100
Actions
Copy link
 #4

Updated by Ward Vandewege almost 11 years ago

Review comments:

1. You're introducing whitespace at the end of one line in app/controllers/users_controller.rb

2. otherwise, good to merge

Thanks,
Ward.

Actions
Copy link
 #5

Updated by Phil Hodgson almost 11 years ago

(Whitespace problem will go away - I've changed my text editor settings. The commit that came from was cherry-picked from some earlier work.)

Actions
Copy link

Also available in: Atom PDF