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

feat: Add phabricator integration #373

Merged
merged 2 commits into from
Jul 25, 2018
Merged

feat: Add phabricator integration #373

merged 2 commits into from
Jul 25, 2018

Conversation

dcramer
Copy link
Member

@dcramer dcramer commented Jul 13, 2018

  • Pull in sentry-phabricator
  • Update to IssuePlugin2
  • Add Link Issue support
  • Add assignee support
  • Remove projectPHIDs in favor of tags

@dcramer
Copy link
Member Author

dcramer commented Jul 13, 2018

Nearly good. Need to figure out why 'tags' is broken when we turn it into a multi field.

screen shot 2018-07-13 at 2 48 42 pm

screen shot 2018-07-13 at 2 50 04 pm

@dcramer
Copy link
Member Author

dcramer commented Jul 16, 2018

Blocked on getsentry/sentry#9051

@joshuarli
Copy link
Member

Needs a rebase to at least 9de9e3a to pass CI.

- Pull in sentry-phabricator
- Update to IssuePlugin2
- Add Link Issue support
- Add assignee support
- Remove projectPHIDs in favor of tags
@dcramer
Copy link
Member Author

dcramer commented Jul 16, 2018

Updated to add a bunch of acceptance assertions and make testing just a little bit easier

Copy link
Member

@evanpurkhiser evanpurkhiser left a comment

Choose a reason for hiding this comment

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

Seems fine to me. I'm not too worried about oldstyle plugins

@dcramer
Copy link
Member Author

dcramer commented Jul 25, 2018

Will try to merge today, just need to remove sentry-phabricator from prod in parallel.

@dcramer dcramer merged commit 5946a77 into master Jul 25, 2018
@dcramer dcramer deleted the feat/phabricator branch July 25, 2018 11:27
@iamareebjamal
Copy link

Any ETA when the next version supporting Phabricator will be released? I tried installing the package through git but it failed

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.

4 participants