Page MenuHomePhabricator

Legalpad - add "no one" signature type
ClosedPublic

Authored by btrahan on Feb 17 2015, 7:39 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Apr 27, 2:00 AM
Unknown Object (File)
Apr 2 2025, 1:03 AM
Unknown Object (File)
Mar 24 2025, 7:25 PM
Unknown Object (File)
Mar 15 2025, 12:38 AM
Unknown Object (File)
Mar 14 2025, 6:51 PM
Unknown Object (File)
Feb 2 2025, 5:04 PM
Unknown Object (File)
Feb 2 2025, 7:34 AM
Unknown Object (File)
Jan 31 2025, 11:39 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.