Correct Transaction text when changing Diffusion URI I/O type
ClosedPublic
Actions

Authored by aklapper on Sep 27 2023, 16:51.

Tags

None

Referenced Files

	F3340638: D25443.1743640611.diff
	Wed, Apr 2, 00:36

	F3339818: D25443.1743626864.diff
	Tue, Apr 1, 20:47

	F3333352: D25443.1743534721.diff
	Mon, Mar 31, 19:12

	F3333340: D25443.1743534696.diff
	Mon, Mar 31, 19:11

	F3332785: D25443.1743526299.diff
	Mon, Mar 31, 16:51

	F3331460: D25443.1743507628.diff
	Mon, Mar 31, 11:40

	F3329921: D25443.1743481806.diff
	Mon, Mar 31, 04:30

	F3323659: D25443.1743388219.diff
	Sun, Mar 30, 02:30

Subscribers

Tokens

"Like" token, awarded by valerio.bozzolan.

Details

Reviewers

avivey

Group Reviewers

O1: Blessed Committers

Maniphest Tasks

T15648: Transaction text when changing Diffusion URI I/O type is about Display type instead

Commits

rP03afb97ff20b: Correct Transaction text when changing Diffusion URI I/O type

Summary

After changing the I/O type of a Diffusion repository URI, the transaction history incorrectly shows a message which talks about changing the URI's Display type instead.

Closes T15648

Test Plan

Change the I/O type of a Diffusion repo URI and look at that URIs history.

Diff Detail

Repository

rP Phorge

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

aklapper created this revision.Sep 27 2023, 16:51

Owners added a reviewer: O1: Blessed Committers.Sep 27 2023, 16:51

Herald added subscribers: Cigaryno, Matthew, valerio.bozzolan, tobiaswiese. · View Herald TranscriptSep 27 2023, 16:51

Harbormaster completed remote builds in B854: Diff 1400.Sep 27 2023, 16:51

aklapper requested review of this revision.Sep 27 2023, 16:51

avivey accepted this revision.Sep 27 2023, 20:12

This revision is now accepted and ready to land.Sep 27 2023, 20:12

valerio.bozzolan edited the summary of this revision. (Show Details)Sep 27 2023, 20:46

valerio.bozzolan edited the test plan for this revision. (Show Details)

valerio.bozzolan awarded a token.

Closed by commit rP03afb97ff20b: Correct Transaction text when changing Diffusion URI I/O type. · Explain WhySep 28 2023, 07:20

This revision was automatically updated to reflect the committed changes.

aklapper added a commit: rP03afb97ff20b: Correct Transaction text when changing Diffusion URI I/O type.

Revision Contents
Changeset List

Path

Size

Packages

src/

applications/

repository/

storage/

PhabricatorRepositoryURITransaction.php

2 lines

O1: Blessed Committers

Diff 1401

View Options

src/applications/repository/storage/PhabricatorRepositoryURITransaction.php

Correct Transaction text when changing Diffusion URI I/O typeClosedPublicActions