Page MenuHomePhabricator

Legalpad - add "no one" signature type
ClosedPublic

Authored by btrahan on Feb 17 2015, 7:39 PM.
Tags
None
Referenced Files
F13265244: D11788.diff
Tue, May 28, 2:34 AM
F13264449: D11788.diff
Mon, May 27, 5:11 PM
F13247258: D11788.diff
Thu, May 23, 5:18 PM
F13235363: D11788.id.diff
Tue, May 21, 4:48 AM
F13225748: D11788.diff
Sun, May 19, 5:10 PM
F13207945: D11788.id28415.diff
Thu, May 16, 6:26 AM
F13207944: D11788.id28414.diff
Thu, May 16, 6:26 AM
F13207943: D11788.id28413.diff
Thu, May 16, 6:26 AM
Subscribers

Details

Summary

Fixes T7294. This lets legalpad store other documents that don't need signatures but conceptually belong in legalpad.

Test Plan

made a document with signature type "no one" and it saved. viewed the document and noted no signing UI was present.

Diff Detail

Repository
rP Phabricator
Branch
T7294
Lint
Lint Passed
SeverityLocationCodeMessage
Advicesrc/applications/legalpad/controller/LegalpadDocumentSignController.php:38XHP16TODO Comment
Unit
No Test Coverage
Build Status
Buildable 4519
Build 4533: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

btrahan retitled this revision from to Legalpad - add "no one" signature type.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
epriestley edited edge metadata.
epriestley added inline comments.
src/applications/legalpad/storage/LegalpadDocument.php
136

Maybe put this last in the list, since it'll default on new documents I think but "individuals" is probably the best / most common default?

This revision is now accepted and ready to land.Feb 17 2015, 7:42 PM
btrahan edited edge metadata.

move "no one" to last in dropdown since its least common. (initNewDocument handled the doc type default already, but its nice to put this rare option in the back)

This revision was automatically updated to reflect the committed changes.