fix Vary Subjects option names
ClosedPublic
Actions

Authored by allixsenos on Jun 14 2016, 5:07 PM.

Details

Reviewers

epriestley

Group Reviewers

Blessed Reviewers

Maniphest Tasks

T11148: option names for Vary Subjects are copypasta from Re-Prefix option

Commits

rPfc1c633bae85: fix Vary Subjects option names

Summary

The option names for Vary Subjects are copypasta from the Add "Re:" Prefix option. Fix their names to refer to Vary Subjects instead.

Fixes T11148

Test Plan

Verify option names for Vary Subjects refer to Add "Re:" Prefix before apply. Verify they no longer do after apply.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

allixsenos updated this revision to Diff 38771.Jun 14 2016, 5:07 PM

allixsenos retitled this revision from to fix Vary Subjects option names.

allixsenos updated this object.

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

allixsenos added a reviewer: epriestley.

allixsenos added a task: T11148: option names for Vary Subjects are copypasta from Re-Prefix option.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJun 14 2016, 5:07 PM

Herald added a subscriber: Korvin. · View Herald Transcript

Thanks!

This revision is now accepted and ready to land.Jun 14 2016, 5:08 PM

Closed by commit rPfc1c633bae85: fix Vary Subjects option names (authored by allixsenos, committed by allixsenos). · Explain WhyJun 14 2016, 5:13 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

settings/

setting/

PhabricatorEmailVarySubjectsSetting.php

4 lines

Diff 38772

View Options

src/applications/settings/setting/PhabricatorEmailVarySubjectsSetting.php

fix Vary Subjects option namesClosedPublicActions