-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bugfix FXIOS-8672 Skip private tab data on store #19936
Conversation
privacySettings += [ | ||
BoolSetting(prefs: profile.prefs, | ||
theme: themeManager.currentTheme(for: windowUUID), | ||
prefKey: "settings.closePrivateTabs", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@OrlaM I think there are a few other places with related code we can probably clean up
![Screenshot 2024-04-22 at 1 08 11鈥疨M](https://private-user-images.githubusercontent.com/145381717/324597054-e97940b9-1e96-4685-a385-d5233049a4b8.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MTk2NzEwODYsIm5iZiI6MTcxOTY3MDc4NiwicGF0aCI6Ii8xNDUzODE3MTcvMzI0NTk3MDU0LWU5Nzk0MGI5LTFlOTYtNDY4NS1hMzg1LWQ1MjMzMDQ5YTRiOC5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjQwNjI5JTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI0MDYyOVQxNDE5NDZaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT1lOTAxNjc0OGE4NmNkNTg4NjAxODllNDlkOGRlNDljN2RiNDc5ZTNhOTI0ODg5Nzc1NTg2MGQwMTI5NTcwMTc5JlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCZhY3Rvcl9pZD0wJmtleV9pZD0wJnJlcG9faWQ9MCJ9.Kr_Wb-P-tS_c0qUdaEd7w9nK4d7p7DQlt-KunCEqbaM)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry I should have left a comment about it, there are a bunch of other places too with code that can be simplified. I've intentionally kept this PR as simple as possible so we can release it and then be sure there is no major blow back from users. If we need to roll back it should be very simple in the current state. I'll create a follow up ticket and assign it myself to get back to this after 127 has been live for a few weeks.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ok sounds good. Change LGTM
Client.app: Coverage: 30.94
Generated by 馃毇 Danger Swift against 4963b94 |
馃摐 Tickets
Jira ticket
馃挕 Description
Skip private tab data on store. Private tabs will no longer be restored when restarting Firefox.
Also removed the setting from settings as that is now the default behaviour.
馃摑 Checklist
You have to check all boxes before merging
@Mergifyio backport release/v120
)