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

GKCS-5303 Allow the browser extension to connect to On-premise GitHub environments #21

Merged
merged 3 commits into from
Mar 1, 2024

Conversation

gitkrakel
Copy link
Contributor

@gitkrakel gitkrakel commented Feb 27, 2024

https://gitkraken.atlassian.net/browse/GKCS-5303

Tested with https://ghe.ci.gitkraken.com (running Enterprise v3.11.2).

There are buttons on commit comparison, individual commits, and repo landing pages. (i.e. < > Code tab of a repo).

However for PRs, I noticed there's no < > Code button, so I didn't add any buttons on PR pages (note: this might be because our instance is behaving weirdly right now: https://gitkraken-hq.slack.com/archives/C05BYV78VQB/p1709170626154339)

@gitkrakel gitkrakel self-assigned this Feb 27, 2024
Copy link
Contributor

@jdgarcia jdgarcia left a comment

Choose a reason for hiding this comment

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

actually, do you mind splitting the enterprise stuff out to a separate PR? the permission stuff by itself is useful and needed, and could be released before the enterprise stuff

package.json Outdated Show resolved Hide resolved
src/types.ts Outdated Show resolved Hide resolved
@gitkrakel
Copy link
Contributor Author

FYI I split the permission handling portion of this into #22

@gitkrakel gitkrakel force-pushed the GKCS-5303-allow-the-browser-extension-to branch 6 times, most recently from 9a72116 to 89c5618 Compare February 29, 2024 16:42
@gitkrakel gitkrakel force-pushed the GKCS-5303-allow-the-browser-extension-to branch from 89c5618 to 219b28a Compare March 1, 2024 21:09
@gitkrakel gitkrakel marked this pull request as ready for review March 1, 2024 21:19
@gitkrakel gitkrakel requested a review from jdgarcia March 1, 2024 21:19
@gitkrakel gitkrakel merged commit b89b861 into main Mar 1, 2024
1 check passed
@gitkrakel gitkrakel deleted the GKCS-5303-allow-the-browser-extension-to branch March 1, 2024 23:41
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.

2 participants