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

fix(onboarding): update the condition to support users updated from v8 #1956

Merged
merged 3 commits into from
Oct 14, 2024

Conversation

smalluban
Copy link
Collaborator

@smalluban smalluban commented Oct 14, 2024

The function uses only the onboarding scope to avoid too many executions when options change.

I've checked and setup_shown is set to 1 and not cleared in v8.

The update process does not open the onboarding anymore.

@smalluban smalluban requested a review from chrmod October 14, 2024 08:50
Copy link
Collaborator Author

@smalluban smalluban left a comment

Choose a reason for hiding this comment

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

As the updated users have options.onboarding.shownAt === 0, I think it might be worth to clear out that option at all. It was cleared out in all v8 versions, @chrmod?

@smalluban smalluban changed the title fix(onboarding): change condition to support users updated from v8 fix(onboarding): update the condition to support users updated from v8 Oct 14, 2024
@smalluban smalluban merged commit e039572 into main Oct 14, 2024
2 checks passed
@smalluban smalluban deleted the fix-onboarding branch October 14, 2024 09:51
@smalluban smalluban mentioned this pull request Oct 14, 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