Fix Up Paste transactions
ClosedPublic
Actions

Authored by chad on Aug 7 2015, 1:46 AM.

Details

Reviewers

Maniphest Tasks

Commits

Restricted Diffusion Commit
Restricted Diffusion Commit
rP58b622852f37: Fix Up Paste transactions
rP2c2eeed65d68: Fix Paste transactions

Summary

We're using the wrong constants here, bad copy and paste job. Also add colors, icons.

Test Plan

Archive a Paste, check transactions.

Diff Detail

Repository

rP Phabricator

Branch

master

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 7518
Build 8076: [Placeholder Plan] Wait for 30 Seconds
Build 8075: arc lint + arc unit

Event Timeline

chad updated this revision to Diff 33366.Aug 7 2015, 1:46 AM

chad retitled this revision from to Fix Up Paste transactions.

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 7 2015, 1:46 AM

epriestley accepted this revision.Aug 7 2015, 1:52 AM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Aug 7 2015, 1:52 AM

chad added a task: T9076: Archiving pastes.Aug 7 2015, 1:54 AM

Closed by commit rP2c2eeed65d68: Fix Paste transactions (authored by chad, committed by chad). · Explain WhyAug 7 2015, 2:01 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

paste/

storage/

PhabricatorPasteTransaction.php

34 lines

Diff 33366

View Options

src/applications/paste/storage/PhabricatorPasteTransaction.php

Fix Up Paste transactionsClosedPublicActions