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

Enhancement: Add support for GitHub enterprise #43

Open
tomquist opened this issue Feb 14, 2019 · 4 comments
Open

Enhancement: Add support for GitHub enterprise #43

tomquist opened this issue Feb 14, 2019 · 4 comments
Labels
enhancement New feature or request

Comments

@tomquist
Copy link

The Plugin currently only works with github.com but it would be nice if it also worked with a custom GitHub enterprise instance.

@alexmdodge alexmdodge added the enhancement New feature or request label Feb 14, 2019
@alexmdodge
Copy link
Owner

Definitely would be possible. I'd have to get my hands on the trial subscription to see what the DOM structure looks like. If they maintain parity between their UI then it would just be the work required to allow people to authorize their particular domain. When I get some time I'll look into what it would be like to get it working.

@alexmdodge alexmdodge changed the title Add support for GitHub enterprise Enhancement: Add support for GitHub enterprise Jul 27, 2020
@alexmdodge
Copy link
Owner

@tomquist A little bit delayed but I've been making some upgrades lately and have had some thoughts around the best way to accomplish this.

There are generally a complications around adding broad permissions, and to support enterprise, they would have to be extremely broad to account for all potential hosting options. Instead I'm wondering if adding a generate approach where you can more easily build the extension locally and provide the custom URL of your enterprise instance would work better.

For instance I could provide something like the following:

npx github-diff-exporer generate --enterprise --host=custom.github.com

Assuming that all internal paths remain the same for any hosted instance, then that could be loaded locally and should work out of the box.

@tomquist
Copy link
Author

@alexmdodge Would definitely be a solution, however not that convenient. The "Better Pull Requests for GitHub" extension works for GitHub enterprise too. I'm not an expert in chrome extensions but I think you could declare optional_permissions like they do and provide a button to request permission for accessing the page.

@alexmdodge
Copy link
Owner

@tomquist Thanks for the links, I'll check those out.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants