-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Remove .nativePhotoPicker feature flag #21681
Conversation
19495e8
to
fa5e9a6
Compare
Generated by 🚫 dangerJS |
📲 You can test the changes from this Pull Request in WordPress Alpha by scanning the QR code below to install the corresponding build.
|
📲 You can test the changes from this Pull Request in Jetpack Alpha by scanning the QR code below to install the corresponding build.
|
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.
Hey @kean, when I try to update my site icon, some times it doesn't update to the new photo. You see the spinner over the old icon, then the spinner goes away. If you pull to refresh, it shows the new photo.
I'm not sure if the device version matters here - I can reproduce this pretty consistently on my iPhone 14 Pro on iOS 16.6.1. I can't seem to reproduce this on the simulator (iPhone 15 Pro, iOS 17)
@momo-ozawa , I opened a bug #21735. It doesn't seem to be related to the changes from this PR. |
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! Can you add release notes for this? Thanks! 🙇♀️
fa5e9a6
to
b5a1209
Compare
Done, thank you. |
Remove
.nativePhotoPicker
feature flag and the associated code.To test:
Requires smoke tests. Please, follow the main steps from the following PRs:
Regression Notes
PR submission checklist:
RELEASE-NOTES.txt
if necessary.UI Changes testing checklist: