Skip to content

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

Merged
ComputelessComputer merged 2 commits intomainfrom
c-branch-1
Mar 16, 2026
Merged

fix: defer onboarding-time restarts until setup is complete#4580
ComputelessComputer merged 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 1602c46
🔍 Latest deploy log https://app.netlify.com/projects/hyprnote-storybook/deploys/69b7d7aefdd2290008afc9f2

@netlify
Copy link

netlify bot commented Mar 14, 2026

Deploy Preview for hyprnote failed.

Name Link
🔨 Latest commit 1602c46
🔍 Latest deploy log https://app.netlify.com/projects/hyprnote/deploys/69b7d7aed1048f00085939b3

@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.

@ComputelessComputer ComputelessComputer force-pushed the c-branch-1 branch 2 times, most recently from 296c19d to 5469f42 Compare March 16, 2026 06:51
- 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
@ComputelessComputer ComputelessComputer merged commit 5e3e998 into main Mar 16, 2026
7 of 13 checks passed
@ComputelessComputer ComputelessComputer deleted the c-branch-1 branch March 16, 2026 10:27
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