Skip to content
This repository has been archived by the owner on Sep 23, 2023. It is now read-only.

Post links to Phabricator #145

Merged
merged 2 commits into from Oct 18, 2020
Merged

Post links to Phabricator #145

merged 2 commits into from Oct 18, 2020

Conversation

edg2s
Copy link
Collaborator

@edg2s edg2s commented Oct 13, 2020

Fixes #144

@edg2s
Copy link
Collaborator Author

edg2s commented Oct 13, 2020

Tested this locally with a key for my own account: https://phabricator.wikimedia.org/T265358#6539089

Just need to create a bot account and generate a key for it.

@edg2s edg2s requested a review from MatmaRex October 13, 2020 15:49
@@ -293,6 +293,26 @@ function set_progress( int $pc, string $label ) {
abandon( "Could not deduplicate." );
}

if ( count( $linkedTasks ) && $config['conduitApiKey'] ) {
set_progress( 95, 'Posting to Phabricator...' );
$api = new \Phabricator\Phabricator( 'https://phabricator.wikimedia.org', $config['conduitApiKey'] );
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should probably move our phab and gerrit URLs to config at some point.

@edg2s
Copy link
Collaborator Author

edg2s commented Oct 14, 2020

Tested with PDBot's key: https://phabricator.wikimedia.org/T265358#6542530 and added that key to live config.

@edg2s
Copy link
Collaborator Author

edg2s commented Oct 15, 2020

New checkbox (selected by default):
image

@MatmaRex MatmaRex merged commit f90d277 into master Oct 18, 2020
@edg2s edg2s deleted the phab-bot branch October 23, 2020 09:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Automatically announce new wikis on Phabricator and/or Gerrit
2 participants