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

Opened up a Codespace with a GH PR and was asked to sign in to GHES #4056

Closed
TylerLeonhardt opened this issue Oct 14, 2022 · 6 comments · Fixed by #4138
Closed

Opened up a Codespace with a GH PR and was asked to sign in to GHES #4056

TylerLeonhardt opened this issue Oct 14, 2022 · 6 comments · Fixed by #4138
Assignees
Labels
bug Issue identified by VS Code Team member as probable bug verified Verification succeeded
Milestone

Comments

@TylerLeonhardt
Copy link
Member

TylerLeonhardt commented Oct 14, 2022

I think this is because I have settings sync turned on and it picked up the GHES setting from when I was testing it out.

image

I don't think this should happen in Codespaces at all.

@alexr00
Copy link
Member

alexr00 commented Oct 17, 2022

I think the GHES setting should probably a workspace only setting. @TylerLeonhardt what do you think?

@TylerLeonhardt
Copy link
Member Author

TylerLeonhardt commented Oct 23, 2022

@alexr00 I actually can't make it a workspace yet because I don't store the GHES uri with the session at the moment. Actually storing the GHES uri along with the session would probably be help in #3992 (comment) as well

@TylerLeonhardt
Copy link
Member Author

TylerLeonhardt commented Oct 31, 2022

@alexr00 as discussed we can't actually make something "workspace setting" only... so we'll need to handle this somehow when the user has this setting set to not prompt.

@alexr00
Copy link
Member

alexr00 commented Oct 31, 2022

Hmm. Does it make sense to set this setting to not sync? I can also prioritize the "is github.com" check over the "has enterprise uri" check.

@alexr00 alexr00 added this to the November 2022 milestone Oct 31, 2022
@TylerLeonhardt
Copy link
Member Author

Does it make sense to set this setting to not sync?

Can I do that while still allowing it to be set in workspace settings? I don't think so... @sandy081?

@alexr00 alexr00 added the bug Issue identified by VS Code Team member as probable bug label Oct 31, 2022
@sandy081
Copy link
Member

sandy081 commented Nov 1, 2022

Settings scoped to machine or machine-overridable are not synced by default.

@connor4312 connor4312 added the author-verification-requested Issues potentially verifiable by issue author label Nov 30, 2022
@TylerLeonhardt TylerLeonhardt added verified Verification succeeded and removed author-verification-requested Issues potentially verifiable by issue author labels Dec 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issue identified by VS Code Team member as probable bug verified Verification succeeded
Projects
None yet
4 participants