Skip to content

Remove unused code related to Avatar upload #21952

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

Merged
merged 1 commit into from
Jun 24, 2025
Merged

Conversation

AdamGrzybkowski
Copy link
Contributor

Description

This PR removes the FF that controlled the Quick Editor. With the flag gon,e we can remove more code that was there to select and upload avatars.

Testing instructions

The FF was enabled for some time so technically we should be good to go, but it won't hurt to make sure it all still works as intended. Having a good look at the removed code to make sure I haven't removed to much would be great as well :D

MeFragment

  1. Test the Quick Editro avatar upload, selection etc.

SignupEpilogueFragment
This is where I would need some help. I remember having a way to enter this screen with the old magic link sign-up flow, but now with the OAuth, I'm not sure how to do it.

Based on what I was able to find, the app opens the LoginEpilogueActivity, not the SignupEpilogueActivity, after I create a new account.

@dangermattic
Copy link
Collaborator

dangermattic commented Jun 13, 2025

2 Warnings
⚠️ strings.xml files should only be updated on release branches, when the translations are downloaded by our automation.
⚠️ This PR is larger than 300 lines of changes. Please consider splitting it into smaller PRs for easier and faster reviews.

Generated by 🚫 Danger

@wpmobilebot
Copy link
Contributor

wpmobilebot commented Jun 13, 2025

Jetpack📲 You can test the changes from this Pull Request in Jetpack by scanning the QR code below to install the corresponding build.
App NameJetpack Jetpack
FlavorJalapeno
Build TypeDebug
Versionpr21952-96f156f
Commit96f156f
Direct Downloadjetpack-prototype-build-pr21952-96f156f.apk
Note: Google Login is not supported on these builds.

@wpmobilebot
Copy link
Contributor

wpmobilebot commented Jun 13, 2025

WordPress📲 You can test the changes from this Pull Request in WordPress by scanning the QR code below to install the corresponding build.
App NameWordPress WordPress
FlavorJalapeno
Build TypeDebug
Versionpr21952-96f156f
Commit96f156f
Direct Downloadwordpress-prototype-build-pr21952-96f156f.apk
Note: Google Login is not supported on these builds.

@nbradbury nbradbury self-assigned this Jun 13, 2025
@AdamGrzybkowski AdamGrzybkowski added this to the 26.0 milestone Jun 17, 2025
@AdamGrzybkowski AdamGrzybkowski marked this pull request as ready for review June 17, 2025 08:02
@nbradbury
Copy link
Contributor

This kind of thing always makes me happy :)

Screenshot 2025-06-17 at 10 47 57 AM

@nbradbury
Copy link
Contributor

@AdamGrzybkowski I'm also not sure how SignupEpilogueActivity fits into the login flow, so I just added this at the end of WPMainActivity.onCreate to force it to appear:

ActivityLauncher.showSignupEpilogue(this, null, null, null, null, true);

To get it to work, though, I had to comment out the mUnifiedLoginTracker calls in SignupEpilogueFragment.

When the epilogue activity appeared my avatar was blank, but tapping to change it worked as expected.

signup

I'm not sure if the blank avatar is due to how I'm forcing the activity to appear or if it's a genuine issue.

Base automatically changed from adam/gravatar_2.5.0 to trunk June 23, 2025 15:32
This commit removes the FF that controled the Quick Editor. With the flag gone we can remove more code that was there to select and upload avatars.
@AdamGrzybkowski
Copy link
Contributor Author

AdamGrzybkowski commented Jun 24, 2025

Thanks!

I'm not sure if the blank avatar is due to how I'm forcing the activity to appear or if it's a genuine issue.

Quick Editor doesn't load the avatar, so it's unlikely to be related. I will take a look, though.

EDIT: The avatar doesn't load because you have to pass the URL via one of the params in the ActivityLauncher.showSignupEpilogue(). So now that you pass null, there's nothing to load.

@AdamGrzybkowski
Copy link
Contributor Author

@nbradbury The other PR updating the Gravatar SDK to 2.5.0 was merged, so I've rebased on top of trunk and this is ready for review.

Copy link

Copy link
Contributor

@nbradbury nbradbury left a comment

Choose a reason for hiding this comment

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

Changes look good, merge when ready! :shipit:

@AdamGrzybkowski AdamGrzybkowski merged commit 8e83e61 into trunk Jun 24, 2025
26 checks passed
@AdamGrzybkowski AdamGrzybkowski deleted the adam/code_cleanup branch June 24, 2025 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Gravatar Gravatar integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants