-
Notifications
You must be signed in to change notification settings - Fork 927
Fix an issue where localCache is not copied as part of Settings #7385
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
Conversation
🦋 Changeset detectedLatest commit: 2a2b57b The changes in this PR will be included in the next version bump. This PR includes changesets to release 3 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
8ee1650
to
3ea1c10
Compare
Size Report 1Affected Products
Test Logs |
Size Analysis Report 1Affected Products
Test Logs |
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.
LGTM. As far as I can tell, there's no public API change. Is that right?
Yep, that is right. |
@markarndt Please review the release note, there is also some minor fixes on public docs, some non-existent classes were incorrectly mentioned in the public ref doc. |
@wu-hui I apologize for mentioning you in a closed pull request. I noticed this change in the recent v10.0.0 changelog, and I'm having a bit of difficulty understanding the full impact of this bug. Could you kindly clarify whether this means the |
The issue is if you call |
Fixes: #7360
Also fixes some public api to use the right classes