Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Conversation

@Sarke
Copy link
Contributor

@Sarke Sarke commented Sep 19, 2024

Another Flatpak/KDE5 wallet workaround, it worked well for me. I think this is the more correct way to do it.

Another Flatpak/KDE5 wallet workaround
@ntrogh ntrogh closed this Sep 19, 2024
@Sarke
Copy link
Contributor Author

Sarke commented Sep 20, 2024

Any reason for just closing it?

@Sarke
Copy link
Contributor Author

Sarke commented Sep 20, 2024

I had the same problem, and searching found this: #6577

And this: microsoft/vscode#189672

I ended up using the permission settings for Flatpak instead of a workaround through the Gnome library, so I thought I'd share.

@ntrogh ntrogh reopened this Sep 20, 2024
@ntrogh
Copy link
Contributor

ntrogh commented Sep 20, 2024

Any reason for just closing it?

@Sarke apologies, I inadvertently closed the PR! Back to reviewing it and pressing the right buttons.

Appreciate your contribution!

@ntrogh
Copy link
Contributor

ntrogh commented Sep 20, 2024

@TylerLeonhardt Can you take a quick look at this docs update? LGTM.

Copy link
Contributor

@ntrogh ntrogh left a comment

Choose a reason for hiding this comment

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

@Sarke Suggest to alter the language to make it clear that this fix is not verified by the team. Thanks a lot for contributing and helping others be successful!

docs/editor/settings-sync.md Outdated Show resolved Hide resolved
Co-authored-by: Nick Trogh <1908215+ntrogh@users.noreply.github.com>
Copy link
Contributor

@ntrogh ntrogh left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks a lot @Sarke

@ntrogh ntrogh merged commit de1ed4d into microsoft:main Sep 23, 2024
@vs-code-engineering vs-code-engineering bot added this to the September 2024 milestone Sep 23, 2024
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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.