Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GitHub Integration v1 #759

Merged
merged 13 commits into from
Apr 22, 2021
Merged

GitHub Integration v1 #759

merged 13 commits into from
Apr 22, 2021

Conversation

reichert621
Copy link
Collaborator

@reichert621 reichert621 commented Apr 20, 2021

Description

This PR will:

  • Set up the github_authorization table
  • Set up a GitHub API client
  • Handle both OAuth and App installs
  • Set up the UX for authorizing Github
  • Set up a webhook URL for handling events
  • Set up the issues table
  • Support linking a Papercups issue to a GitHub issue
  • Support updating a Papercups issue's state based on the GitHub issue state

TODOs for followup PRs:

  • User can send emails to customers from customer details page
  • User can select main GitHub repo to link to Papercups (since GitHub supports multiple by default)
  • User can create new issues in GitHub directly from Papercups
  • User can search GitHub issues when linking Papercups issue to GitHub (rather than pasting in GitHub URL)
  • User can perform more actions from Papercups Issues page (e.g. update state)
  • User can edit Papercups Issues

Issue

#745
#749

Screenshots

Authorization flow:
github-auth

Create new Papercups issue:
create-new-issue

Link issue to existing customer:
link-issue-to-customer

Link customer to existing issue:
link-customer-to-issue

Sync state with GitHub:
sync-github-state

Checklist

  • Everything passes when running mix test
  • Ran mix format
  • No frontend compilation warnings

@reichert621 reichert621 force-pushed the github-integration-v1 branch from 5b51178 to d8e3370 Compare April 20, 2021 18:16
* Set up boilerplate for github_authorizations

* Clena up GithubAuthorizationView

* Support JWT auth for github, add auth flow to UI

* Confirm before disconnect
* Set up boilerplate for Issues model

* Add basic UI for issues

* Minor cleanup
@reichert621 reichert621 force-pushed the github-integration-v1 branch from 37fd5f5 to a4496ba Compare April 21, 2021 14:09
@reichert621 reichert621 changed the title GitHub Integration v1 (WIP) GitHub Integration v1 Apr 22, 2021
@reichert621 reichert621 marked this pull request as ready for review April 22, 2021 17:52
@reichert621 reichert621 merged commit d244476 into master Apr 22, 2021
@reichert621 reichert621 deleted the github-integration-v1 branch April 22, 2021 20:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant