Skip to content

[dashboard] github app installation #3704

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

Merged
merged 1 commit into from
Apr 1, 2021

Conversation

svenefftinge
Copy link
Member

@svenefftinge svenefftinge commented Apr 1, 2021

How to Test

@svenefftinge svenefftinge force-pushed the sefftinge/implement-install-github-3702 branch 6 times, most recently from 12f11c0 to bdb4f3e Compare April 1, 2021 16:42
@svenefftinge svenefftinge marked this pull request as ready for review April 1, 2021 16:49
@svenefftinge
Copy link
Member Author

TODO: Remove changes in values.dev.yaml before merge

@svenefftinge
Copy link
Member Author

fixes ##3702

@@ -1205,7 +1205,7 @@ export class GitpodServerImpl<Client extends GitpodClient, Server extends Gitpod
const user = this.checkAndBlockUser();

if (!this.env.githubAppEnabled) {
throw new ResponseError(ErrorCodes.NOT_FOUND, 'User is not authenticated. Please login.');
throw new ResponseError(ErrorCodes.NOT_FOUND, 'No GitHub enabled for this installation. Please talk to your administrator.');
Copy link
Member Author

Choose a reason for hiding this comment

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

Should be 'No GitHub app enabled ....'

Copy link
Contributor

@csweichel csweichel left a comment

Choose a reason for hiding this comment

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

LGTM

@svenefftinge svenefftinge force-pushed the sefftinge/implement-install-github-3702 branch from bdb4f3e to 2250b76 Compare April 1, 2021 17:49
@svenefftinge svenefftinge force-pushed the sefftinge/implement-install-github-3702 branch from 2250b76 to 55b137c Compare April 1, 2021 17:50
@svenefftinge svenefftinge merged commit 89e8374 into main Apr 1, 2021
@svenefftinge svenefftinge deleted the sefftinge/implement-install-github-3702 branch April 1, 2021 17:57
<span className="text-gray-500">This action will also allow Gitpod to access private repositories. You can edit git provider permissions later in user settings.</span>
</div>
<div className="mt-6">
<button className="secondary">Cancel</button>
Copy link
Contributor

@gtsiolis gtsiolis Apr 1, 2021

Choose a reason for hiding this comment

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

issue: This went through with no action. We could potentially remove the button all together. /cc @csweichel @svenefftinge

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.

3 participants