-
Notifications
You must be signed in to change notification settings - Fork 1.5k
Bitbucket Data Center cloud integration #4146
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
Conversation
a1dc12f to
ccdc77f
Compare
4e71b03 to
aa24560
Compare
|
|
||
| protected get apiBaseUrl(): string { | ||
| const protocol = this._session?.protocol ?? 'https:'; | ||
| return `${protocol}//${this.domain}/rest/api/1.0`; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @ericf-axosoft, @axosoft-ramint
URL of our test server is on HTTP, that's why I needed to make an update to gather the protocol from Cloud. These changes are here: a7a792a
a7a792a to
1449867
Compare
cca8e86 to
6ad28b2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can use the shared library normalizers later when they are exposed.
…ntegrations on the Home View (gitkraken#4107, gitkraken#4146)
Description
solves #4107
Checklist
Fixes $XXX -orCloses #XXX -prefix to auto-close the issue that your PR addresses