Page MenuHomePhabricator

Fixed typo on Releeph product query dropdown.
ClosedPublic

Authored by CodeMouse92 on May 18 2015, 5:39 PM.
Tags
None
Referenced Files
F13232262: D12904.diff
Tue, May 21, 1:09 AM
F13225375: D12904.id31110.diff
Sun, May 19, 2:56 PM
F13216060: D12904.diff
Fri, May 17, 10:05 PM
F13209238: D12904.id31111.diff
Thu, May 16, 10:46 PM
F13187029: D12904.diff
Sat, May 11, 4:17 AM
Unknown Object (File)
Tue, May 7, 7:22 AM
Unknown Object (File)
Fri, May 3, 6:01 AM
Unknown Object (File)
Tue, Apr 30, 2:40 AM
Subscribers

Details

Summary

Changed "Active Prodcuts" to "Active Products" on Releeph Product Query dropdown.

(Incidentally, this is my first diff on Phabricator proper, so I hope I got it right. I'm used to SVN, not Git.)

Test Plan

Make sure the change is there and didn't bork anything else.

Diff Detail

Repository
rP Phabricator
Branch
master
Lint
Lint Errors
SeverityLocationCodeMessage
Errorsrc/applications/auth/provider/PhabricatorAuthProvider.php:18PHL1Unknown Symbol
Errorsrc/applications/celerity/CelerityResourceGraph.php:10PHL1Unknown Symbol
Errorsrc/applications/config/option/PhabricatorConfigOption.php:125PHL1Unknown Symbol
Errorsrc/applications/config/schema/PhabricatorConfigSchemaQuery.php:14PHL1Unknown Symbol
Errorsrc/applications/daemon/view/PhabricatorDaemonLogEventsView.php:30PHL1Unknown Symbol
Errorsrc/applications/daemon/view/PhabricatorDaemonLogListView.php:17PHL1Unknown Symbol
Errorsrc/applications/differential/view/DifferentialLocalCommitsView.php:22PHL1Unknown Symbol
Errorsrc/applications/differential/view/DifferentialRevisionListView.php:62PHL1Unknown Symbol
Errorsrc/applications/diffusion/query/DiffusionLintCountQuery.php:26PHL1Unknown Symbol
Errorsrc/applications/diffusion/query/lowlevel/DiffusionLowLevelQuery.php:20PHL1Unknown Symbol
Errorsrc/applications/diviner/atom/DivinerAtom.php:98PHL1Unknown Symbol
Errorsrc/applications/doorkeeper/view/DoorkeeperTagView.php:15PHL1Unknown Symbol
Errorsrc/applications/files/controller/PhabricatorFileDataController.php:230PHL1Unknown Symbol
Errorsrc/applications/fund/phortune/FundBackerCart.php:36PHL1Unknown Symbol
Errorsrc/applications/multimeter/data/MultimeterControl.php:91PHL1Unknown Symbol
Errorsrc/applications/paste/view/PasteEmbedView.php:31PHL1Unknown Symbol
Errorsrc/applications/phid/query/PhabricatorObjectQuery.php:81PHL1Unknown Symbol
Errorsrc/applications/pholio/view/PholioMockEmbedView.php:20PHL1Unknown Symbol
Errorsrc/applications/phortune/cart/PhortuneSubscriptionCart.php:37PHL1Unknown Symbol
Errorsrc/applications/search/controller/PhabricatorApplicationSearchController.php:58PHL1Unknown Symbol
Errorsrc/applications/search/engine/PhabricatorApplicationSearchEngine.php:36PHL1Unknown Symbol
Errorsrc/applications/slowvote/view/SlowvoteEmbedView.php:25PHL1Unknown Symbol
Errorsrc/applications/subscriptions/editor/PhabricatorSubscriptionsEditor.php:59PHL1Unknown Symbol
Errorsrc/applications/transactions/editor/PhabricatorApplicationTransactionCommentEditor.php:136PHL1Unknown Symbol
Errorsrc/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php:981PHL1Unknown Symbol
Unit
No Test Coverage
Build Status
Buildable 6072
Build 6092: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

CodeMouse92 retitled this revision from to Fixed typo on Releeph product query dropdown..
CodeMouse92 updated this object.
CodeMouse92 edited the test plan for this revision. (Show Details)
CodeMouse92 added reviewers: chad, epriestley.
epriestley edited edge metadata.

Follow the instructions in the lint message to clear lint. They are your fault, and the very first part of the message explains how to resolve them.

This revision now requires changes to proceed.May 19 2015, 1:52 PM
CodeMouse92 edited edge metadata.

Added shebang to file, as requested...?

epriestley edited edge metadata.

Sorry, get rid of the shebang. The shebang warning is because the file is +x in your environment for some reason; it should not be. It is not +x in the repository.

Anyway, this is close enough. I'll fix it in the pull.

This revision is now accepted and ready to land.May 19 2015, 4:59 PM

Ah, thanks for the clarification. I knew something wasn't right.

(I'm near totally unfamiliar with Git. I always use SVN.) I ran arc land 'master' as instructed at the top, and it failed with

You can not land a branch onto itself -- you are trying to land 'master' onto 'master'. For more information on how to push changes, see 'Pushing and Closing Revisions' in 'Arcanist User Guide: arc diff' in the documentation. You may be able to 'arc amend' instead.

So, I ran arc amend, and that said

Amending commit message to reflect revision D12904: Fixed typo on Releeph product query dropdown..

...but did not apparently commit. Repeated both, same results. The Arcanist User Guide tells me nothing additional. I'm lost.

This revision was automatically updated to reflect the committed changes.

You don't have commit rights to the repository. I've pulled the change, I just had a few other things come up.

Ah, okay, well that explains that. I'm still getting my bearings around here, as my own workflows are muscle memory. Thank you!