Page MenuHomePhabricator

Update Legalpad to use modular transactions
ClosedPublic

Authored by chad on May 4 2017, 4:57 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Dec 16, 2:00 AM
Unknown Object (File)
Sun, Dec 8, 12:49 PM
Unknown Object (File)
Thu, Dec 5, 10:16 AM
Unknown Object (File)
Tue, Nov 26, 9:03 PM
Unknown Object (File)
Nov 21 2024, 11:05 PM
Unknown Object (File)
Nov 21 2024, 4:34 PM
Unknown Object (File)
Nov 18 2024, 7:53 PM
Unknown Object (File)
Nov 18 2024, 5:39 AM
Subscribers

Details

Summary

Update Legalpad for modular transactions

Test Plan
  • New Document (no sign)
  • New Document (individual)
  • New Document (corp)
  • Require Signature - get prompted to sign before I can do anything.
  • Edit Documents
  • Sign Documents
  • Comment on Documents

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

src/applications/legalpad/editor/LegalpadDocumentEditor.php
30

Set a document you haven't signed to "Require signature to use Phabricator."

If the code works, you'll immediately be prompted to sign it.

The meaning here is "when a new document is configured to require a signature, invalidate the flag on all existing sessions so users have to sign it even if they were previously logged in with valid sessions".

src/applications/legalpad/storage/LegalpadTransaction.php
20–21

(Might not get called / be unnecessary?)

src/applications/legalpad/xaction/LegalpadDocumentTitleTransaction.php
30

Word "document" probably reads awkwardly; remove?

This all looks fine, just kicking it back to your queue for that TODO.

This revision now requires changes to proceed.May 4 2017, 6:01 PM
chad edited edge metadata.
chad marked 3 inline comments as done.
  • move session destruction
This revision is now accepted and ready to land.May 4 2017, 7:32 PM
This revision was automatically updated to reflect the committed changes.