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)
Tue, Jun 17, 4:18 AM
Unknown Object (File)
Mon, Jun 16, 6:09 PM
Unknown Object (File)
Sun, Jun 15, 6:09 PM
Unknown Object (File)
May 3 2025, 11:58 PM
Unknown Object (File)
Apr 29 2025, 6:22 PM
Unknown Object (File)
Apr 28 2025, 1:07 AM
Unknown Object (File)
Apr 25 2025, 8:41 AM
Unknown Object (File)
Mar 27 2025, 6:04 PM
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.