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.

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

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

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

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

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?

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..

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

Path

Size

Packages

src/

infrastructure/

env/

PhabricatorEnv.php

2 lines

O1: Blessed Committers

Diff 44

View Options

src/infrastructure/env/PhabricatorEnv.php

(PhabricatorENV) update doclinks to link to we.phorge.it instead of secure.phabricator.comClosedPublicActions