Page MenuHomePhabricator
Differential D19598

Condition "Changes Since Last Action" Differential link on "first broadcast", not "new object"
ClosedPublic
Actions

Authored by epriestley on Aug 24 2018, 4:52 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mar 19 2024, 4:20 PM
Unknown Object (File)
Feb 12 2024, 10:24 PM
Unknown Object (File)
Jan 25 2024, 11:47 PM
Unknown Object (File)
Dec 23 2023, 7:41 AM
Unknown Object (File)
Dec 22 2023, 12:14 AM
Unknown Object (File)
Dec 5 2023, 3:41 PM
Unknown Object (File)
Nov 30 2023, 12:47 AM
Unknown Object (File)
Nov 25 2023, 7:50 PM
View All 63 Files
Subscribers
None

Details

Reviewers
amckinley
Maniphest Tasks
T13176: Differential now has a "/D123/new/" link
T13187: Plans: 2018 Week 34 Bonus Content
Commits
rP5e4d9dfa9219: Condition "Changes Since Last Action" Differential link on "first broadcast"…
Summary

Ref T13187. Ref T13176. With drafts, we actually want to suppress this link on "first broadcast" (the first time we send mail), not on "new object" (when the revision is created as a draft).

Test Plan

Poked at this locally, will keep an eye on it in production.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Aug 24 2018, 4:52 PM
Harbormaster completed remote builds in B20657: Diff 46856.Aug 24 2018, 4:53 PM
epriestley requested review of this revision.Aug 24 2018, 4:53 PM
This revision was not accepted when it landed; it landed in state Needs Review.Aug 24 2018, 5:04 PM
Closed by commit rP5e4d9dfa9219: Condition "Changes Since Last Action" Differential link on "first broadcast"… (authored by epriestley). · Explain Why
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
differential/
editor/
2 lines

Diff 46858

View Options

src/applications/differential/editor/DifferentialTransactionEditor.php

Loading...
Phabricator · Built by Phacility