Paths

Table of Contentst

Merge Differential comment migration to ApplicationTransactions
e88d0f9c42d4
Actions

Description

Merge Differential comment migration to ApplicationTransactions

Summary: Ref T2222. This merges the tmp.differential branch, including the
comment -> application transaction migration, to master.

Auditors: btrahan

Details

Provenance

epriestley	Authored on Feb 13 2014, 5:04 PM
themackabu	Pushed on Mar 25 2025, 8:07 PM

Parents

rP7374a0a4d4f5: Fix inline comment links for non-visible comments
rP314331095411: Use pixel as background of timeline

Branches

Unknown

Tags

Unknown

Event Timeline

Merged Changes

7374a0a4d4f5 Fix inline comment links for non-visible comments
Author
epriestley
Browse Repository
No Builds
No Audits
9afe52de51d7 Provide transaction strings, icons and colors for Differential
Author
epriestley
Browse Repository
No Builds
No Audits
cb20205aee10 Don't show "edit" links in Differential for now
Author
epriestley
Browse Repository
No Builds
No Audits
c7ca3cd4abd8 Pass $all_changesets, not $changesets, to inline rendering logic
Author
epriestley
Browse Repository
No Builds
No Audits
ee38e334963b Fix two issues with Differential Transactions
Author
epriestley
Browse Repository
No Builds
No Audits
015931576330 Add some missing text for Differential inline transactions
Author
epriestley
Browse Repository
No Builds
No Audits
348b07a4becf Make Asana bridge work with new Differential transactions
Author
epriestley
Browse Repository
No Builds
No Audits
29918e1bd47f Fix double-rendering of inline comments in mail
Author
epriestley
Browse Repository
No Builds
No Audits
62cb58408346 Use timeline view in Differential and make inlines somewhat usable again
Author
epriestley
Browse Repository
No Builds
No Audits
0dc73d5d9357 Always provide a valid content source for DifferentialCommentEditor
Author
epriestley
Browse Repository
No Builds
No Audits
18938b531071 Migrate Differential comments to ApplicationTransactions
Author
epriestley
Browse Repository
No Builds
No Audits

Changes (21)

Path

Size

resources/

sql/

autopatches/

20140212.dx.1.armageddon.php

src/

__phutil_library_map__.php

applications/

differential/

conduit/

ConduitAPI_differential_getrevisioncomments_Method.php

constants/

DifferentialAction.php

controller/

DifferentialRevisionViewController.php

editor/

DifferentialCommentEditor.php

DifferentialRevisionEditor.php

mail/

DifferentialCommentMail.php

query/

DifferentialCommentQuery.php

search/

DifferentialSearchIndexer.php

storage/

DifferentialComment.php

DifferentialInlineComment.php

DifferentialRevision.php

DifferentialTransaction.php

view/

DifferentialRevisionCommentListView.php

DifferentialTransactionView.php

	differential/	query/
	maniphest/	query/

	DifferentialTransactionQuery.php
	ManiphestTransactionQuery.php

feed/

story/

PhabricatorFeedStoryDifferential.php

pholio/

storage/

PholioTransaction.php

view/

PholioTransactionView.php

releeph/

field/

specification/

ReleephDiffChurnFieldSpecification.php

rPe88d0f9c42d4

View Options

resources/sql/autopatches/20140212.dx.1.armageddon.php

View Options

src/__phutil_library_map__.php

View Options

src/applications/differential/conduit/ConduitAPI_differential_getrevisioncomments_Method.php

View Options

src/applications/differential/constants/DifferentialAction.php

View Options

src/applications/differential/controller/DifferentialRevisionViewController.php

View Options

src/applications/differential/editor/DifferentialCommentEditor.php

View Options

src/applications/differential/editor/DifferentialRevisionEditor.php

View Options

src/applications/differential/mail/DifferentialCommentMail.php

View Options

src/applications/differential/query/DifferentialCommentQuery.php

View Options

src/applications/differential/query/DifferentialTransactionQuery.php

View Options

src/applications/differential/search/DifferentialSearchIndexer.php

View Options

src/applications/differential/storage/DifferentialComment.php

View Options

src/applications/differential/storage/DifferentialInlineComment.php

View Options

src/applications/differential/storage/DifferentialRevision.php

View Options

src/applications/differential/storage/DifferentialTransaction.php

View Options

src/applications/differential/view/DifferentialRevisionCommentListView.php

View Options

src/applications/differential/view/DifferentialTransactionView.php

View Options

src/applications/feed/story/PhabricatorFeedStoryDifferential.php

View Options

src/applications/pholio/storage/PholioTransaction.php

View Options

src/applications/pholio/view/PholioTransactionView.php

View Options

src/applications/releeph/field/specification/ReleephDiffChurnFieldSpecification.php

Merge Differential comment migration to ApplicationTransactionse88d0f9c42d4Actions

Description

Details

Event Timeline

Merged Changes

Changes (21)

rPe88d0f9c42d4

resources/sql/autopatches/20140212.dx.1.armageddon.php

src/__phutil_library_map__.php

src/applications/differential/conduit/ConduitAPI_differential_getrevisioncomments_Method.php

src/applications/differential/constants/DifferentialAction.php

src/applications/differential/controller/DifferentialRevisionViewController.php

src/applications/differential/editor/DifferentialCommentEditor.php

src/applications/differential/editor/DifferentialRevisionEditor.php

src/applications/differential/mail/DifferentialCommentMail.php

src/applications/differential/query/DifferentialCommentQuery.php

src/applications/differential/query/DifferentialTransactionQuery.php

src/applications/differential/search/DifferentialSearchIndexer.php

src/applications/differential/storage/DifferentialComment.php

src/applications/differential/storage/DifferentialInlineComment.php

src/applications/differential/storage/DifferentialRevision.php

src/applications/differential/storage/DifferentialTransaction.php

src/applications/differential/view/DifferentialRevisionCommentListView.php

src/applications/differential/view/DifferentialTransactionView.php

src/applications/feed/story/PhabricatorFeedStoryDifferential.php

src/applications/pholio/storage/PholioTransaction.php

src/applications/pholio/view/PholioTransactionView.php

src/applications/releeph/field/specification/ReleephDiffChurnFieldSpecification.php

Merge Differential comment migration to ApplicationTransactions
e88d0f9c42d4
Actions