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

discussion: what would it take to make this work with enterprise github? #25

Closed
mikehaas763 opened this issue Sep 29, 2017 · 9 comments · Fixed by #27
Closed

discussion: what would it take to make this work with enterprise github? #25

mikehaas763 opened this issue Sep 29, 2017 · 9 comments · Fixed by #27
Assignees
Milestone

Comments

@mikehaas763
Copy link

mikehaas763 commented Sep 29, 2017

This would be really useful in an enterprise environment like mine where we have our own enterprise instance of github

what would it take to make this work with enterprise github?

@chrmarti
Copy link
Contributor

If the remote URL can be used to derive the GHE URL, we could generalize the code here: https://github.com/Microsoft/vscode-github-issues-prs/blob/master/src/github-issues-prs.ts#L358

We use the GitHub NPM module which supports GHE as the example in its README shows: https://github.com/mikedeboer/node-github

Not sure what a good strategy is to decide whether or not to try to connect to a URL as GHE. Maybe ask the user the first time.

Ikuyadeu added a commit to Ikuyadeu/vscode-github-issues-prs that referenced this issue Oct 19, 2017
@chrmarti
Copy link
Contributor

chrmarti commented Nov 2, 2017

@mikehaas763 Do you have access to a GHE install? @Ikuyadeu is working on this in #27 and would need some help with testing it.

@mikehaas763
Copy link
Author

@chrmarti Yes I do at work

@chrmarti
Copy link
Contributor

chrmarti commented Nov 6, 2017

@mikehaas763 I think #27 is ready for testing, could you give it a try?

@mikehaas763
Copy link
Author

@chrmarti yes I'll try it out right now

chrmarti pushed a commit that referenced this issue Nov 8, 2017
@chrmarti chrmarti reopened this Nov 8, 2017
@chrmarti
Copy link
Contributor

chrmarti commented Nov 9, 2017

@mikehaas763 I have published @Ikuyadeu's change with the setting hidden. Could you update the extension to the latest version 0.8.0 and set "github.host" to your GHE host for testing? (You can ignore the settings editor complaining about the setting being unknown.)

@pdavis77
Copy link

worked for me - using fqdn
thx!

@mikehaas763
Copy link
Author

@chrmarti it worked for me as well!

@chrmarti
Copy link
Contributor

Republished with the setting revealed. Thanks @Ikuyadeu for the PR!

@chrmarti chrmarti added this to the November 2017 milestone Nov 15, 2017
kieferrm pushed a commit to kieferrm/vscode-github-issues-prs that referenced this issue Apr 4, 2018
kieferrm pushed a commit to kieferrm/vscode-github-issues-prs that referenced this issue Apr 4, 2018
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 a pull request may close this issue.

3 participants