Skip to content

fix: wait for permissions before initializing messenger #5579

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 2 commits into from
May 9, 2025

Conversation

tomasz-stefaniak
Copy link
Collaborator

@tomasz-stefaniak tomasz-stefaniak commented May 8, 2025

Description

fix: wait for permissions before initializing messenger

Checklist

  • [] I've read the contributing guide
  • [] The relevant docs, if any, have been updated or created
  • [] The relevant tests, if any, have been updated or created

Screenshots

[ For visual changes, include screenshots. Screen recordings are particularly helpful, and appreciated! ]

Testing instructions

[ For new or modified features, provide step-by-step testing instructions to validate the intended behavior of the change, including any relevant tests to run. ]


Summary by mrge

Fixed an issue where the messenger could start before file permissions were set, which could cause errors on some systems.

  • Bug Fixes
    • Now waits for permissions to be set before initializing the messenger process.

@tomasz-stefaniak tomasz-stefaniak requested a review from a team as a code owner May 8, 2025 22:08
@tomasz-stefaniak tomasz-stefaniak requested review from sestinj and removed request for a team May 8, 2025 22:08
@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label May 8, 2025
Copy link

netlify bot commented May 8, 2025

Deploy Preview for continuedev canceled.

Name Link
🔨 Latest commit 6075e7f
🔍 Latest deploy log https://app.netlify.com/sites/continuedev/deploys/681e5e0377a27100087c03dd

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label May 8, 2025
@tomasz-stefaniak tomasz-stefaniak merged commit c2478d4 into main May 9, 2025
34 checks passed
@tomasz-stefaniak tomasz-stefaniak deleted the tomasz/fix-intellij-installation branch May 9, 2025 20:11
@github-project-automation github-project-automation bot moved this from Todo to Done in Issues and PRs May 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm This PR has been approved by a maintainer size:S This PR changes 10-29 lines, ignoring generated files.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants