Skip to content

fix: defer onboarding-time restarts until setup is complete#4580

Open
ComputelessComputer wants to merge 2 commits intomainfrom
c-branch-1
Open

fix: defer onboarding-time restarts until setup is complete#4580
ComputelessComputer wants to merge 2 commits intomainfrom
c-branch-1

Conversation

@ComputelessComputer
Copy link
Collaborator

  • queue automatic app relaunches while onboarding is still required
  • restart after onboarding finishes so system-audio and storage changes still apply
  • add focused tests for the relaunch coordinator

@netlify
Copy link

netlify bot commented Mar 14, 2026

Deploy Preview for hyprnote-storybook canceled.

Name Link
🔨 Latest commit 296c19d
🔍 Latest deploy log https://app.netlify.com/projects/hyprnote-storybook/deploys/69b76a4ece662f00083aa1c2

@netlify
Copy link

netlify bot commented Mar 14, 2026

Deploy Preview for hyprnote ready!

Name Link
🔨 Latest commit 296c19d
🔍 Latest deploy log https://app.netlify.com/projects/hyprnote/deploys/69b76a4ea567a30008eeda50
😎 Deploy Preview https://deploy-preview-4580--hyprnote.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@ComputelessComputer
Copy link
Collaborator Author

/staging

@github-actions
Copy link

Branch Commit Status
c-branch-1 b0812458 View

@ComputelessComputer
Copy link
Collaborator Author

/staging

@github-actions
Copy link

Branch Commit Status
c-branch-1 a54d18b2 View

@ComputelessComputer
Copy link
Collaborator Author

confirming that it works just fine — could change the native alert message to smth like:

Let's restart the app to have permissions applied correctly.

BUT we should check first if this is unnecessary in the first place as @yujonglee pointed out.

- queue automatic app relaunches while onboarding is still required
- restart after onboarding finishes so system-audio and storage changes still apply
- add focused tests for the relaunch coordinator
- delay the system audio restart modal until onboarding finishes
- mark system audio as authorized locally after a successful request
- keep the deferred restart flow intact by showing the modal right before relaunch
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.

1 participant