Page MenuHomePhabricator
Differential D13801

Allow setting of Archive/Active on Paste
ClosedPublic
Actions

Authored by chad on Aug 5 2015, 8:08 PM.
Tags
None
Referenced Files
F14387102: D13801.id33330.diff
Sat, Dec 21, 2:24 PM
Unknown Object (File)
Wed, Nov 27, 4:33 PM
Unknown Object (File)
Sat, Nov 23, 9:25 AM
Unknown Object (File)
Sat, Nov 23, 9:25 AM
Unknown Object (File)
Sat, Nov 23, 9:25 AM
Unknown Object (File)
Nov 19 2024, 4:05 AM
Unknown Object (File)
Nov 18 2024, 4:43 PM
Unknown Object (File)
Nov 12 2024, 1:24 PM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
Maniphest Tasks
T9076: Archiving pastes
Commits
Restricted Diffusion Commit
rP5eeda6040eca: Allow setting of Archive/Active on Paste
Summary

Ref T9076, adds basic plumbing for setting the state of a Paste.

Test Plan

Archive Paste, Activate Paste, New Paste

Diff Detail

Repository
rP Phabricator
Branch
paste-archive
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 7498
Build 8036: [Placeholder Plan] Wait for 30 Seconds
Build 8035: arc lint + arc unit

Event Timeline

chad updated this revision to Diff 33330.Aug 5 2015, 8:08 PM
chad retitled this revision from to Allow setting of Archive/Active on Paste.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
Herald added a subscriber: Korvin. · View Herald TranscriptAug 5 2015, 8:08 PM
chad updated this object.Aug 5 2015, 8:10 PM
chad added a task: T9076: Archiving pastes.
epriestley accepted this revision.Aug 5 2015, 8:38 PM
epriestley edited edge metadata.

I think most applications make this a separate action, but whatever.

This revision is now accepted and ready to land.Aug 5 2015, 8:38 PM
chad added a comment.Aug 5 2015, 8:42 PM

whats that mean?

chad added a comment.Aug 5 2015, 8:42 PM

oh, like an "Action" action

epriestley added a comment.Aug 5 2015, 8:44 PM

Yeah, like you click "Archive" instead of "Edit Thing > Status: Archived".

Closed by commit rP5eeda6040eca: Allow setting of Archive/Active on Paste (authored by chad, committed by chad). · Explain WhyAug 5 2015, 8:44 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
resources/
sql/
autopatches/
2 lines
2 lines
src/
applications/
paste/
controller/
12 lines
12 lines
editor/
8 lines
storage/
17 lines
21 lines

Diff 33330

View Options

resources/sql/autopatches/20150805.paste.status.1.sql

Loading...
View Options

resources/sql/autopatches/20150805.paste.status.2.sql

Loading...
View Options

src/applications/paste/controller/PhabricatorPasteEditController.php

Loading...
View Options

src/applications/paste/controller/PhabricatorPasteViewController.php

Loading...
View Options

src/applications/paste/editor/PhabricatorPasteEditor.php

Loading...
View Options

src/applications/paste/storage/PhabricatorPaste.php

Loading...
View Options

src/applications/paste/storage/PhabricatorPasteTransaction.php

Loading...
Phabricator · Built by Phacility