Page MenuHomePhorge
Differential D25014

(PhabricatorENV) update doclinks to link to we.phorge.it instead of secure.phabricator.com
ClosedPublic
Actions

Authored by MacFan4000 on Jul 23 2021, 23:04.
Tags
None
Referenced Files
F2160640: D25014.id58.diff
Thu, Apr 25, 08:32
Unknown Object (File)
Tue, Apr 23, 16:25
Unknown Object (File)
Sun, Apr 21, 20:17
Unknown Object (File)
Thu, Apr 11, 12:26
Unknown Object (File)
Thu, Apr 11, 09:43
Unknown Object (File)
Thu, Apr 11, 02:13
Unknown Object (File)
Thu, Apr 11, 01:52
Unknown Object (File)
Thu, Apr 11, 01:47
View All Files
Subscribers
speck
taavi
tobiaswiese

Details

Reviewers
deadalnix
Group Reviewers
O1: Blessed Committers
Commits
rPce31289228b9: (PhabricatorENV) update doclinks to link to we.phorge.it instead of secure.
Summary

With this change, links to documentation will point to we.phorge.it rather then secure.phabricator,com

Test Plan

No testing needed - simple URL update

Diff Detail

Repository
rP Phorge
Branch
master
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 32
Build 32: arc lint + arc unit

Event Timeline

MacFan4000 created this revision.Jul 23 2021, 23:04
Owners added a reviewer: O1: Blessed Committers.Jul 23 2021, 23:04
Herald added subscribers: tobiaswiese, speck. · View Herald TranscriptJul 23 2021, 23:04
MacFan4000 requested review of this revision.Jul 23 2021, 23:04
Harbormaster completed remote builds in B32: Diff 44.Jul 23 2021, 23:04
deadalnix accepted this revision.Jul 23 2021, 23:36
This revision is now accepted and ready to land.Jul 23 2021, 23:36
deadalnix added a comment.Jul 23 2021, 23:37

It is a step forward, but considering how the URL could be factorized would be a plus, IMO.

MacFan4000 added a comment.EditedJul 23 2021, 23:46

Thanks for accepting this. I can’t land it though, somebody with the proper permissions will need to do it.

MacFan4000 added a comment.Aug 19 2021, 14:42

Could a member of Blessed Committers please land this? I can't do it my self (no write access) and I would appreciate it.

taavi subscribed.Aug 23 2021, 08:41
In D25014#624, @MacFan4000 wrote:

Could a member of Blessed Committers please land this? I can't do it my self (no write access) and I would appreciate it.

I added you to Trusted Contributors, try now?

MacFan4000 added a comment.EditedAug 23 2021, 12:45

Per https://we.phorge.it/source/phorge/manage/policies/ the push policy for this repo is set to Blessed Committers

Closed by commit rPce31289228b9: (PhabricatorENV) update doclinks to link to we.phorge.it instead of secure.. · Explain WhySep 4 2021, 19:47
This revision was automatically updated to reflect the committed changes.
speck added a commit: rPce31289228b9: (PhabricatorENV) update doclinks to link to we.phorge.it instead of secure..
speck added a comment.Sep 4 2021, 19:47

Thanks for submitting this!

avivey mentioned this in Q16: How to use self-hosted Diviner instead of Phabricator? (Answer 9).Sep 16 2022, 06:08

Revision Contents
Changeset List

PathSizePackages
src/
infrastructure/
env/
2 linesO1: Blessed Committers

Diff 44

View Options

src/infrastructure/env/PhabricatorEnv.php

Loading...
Content licensed under Creative Commons Attribution-ShareAlike 4.0 (CC-BY-SA) unless otherwise noted; code licensed under Apache 2.0 or other open source licenses. · CC BY-SA 4.0 · Apache 2.0