[$8000] Feature Request - Profile / Workspace - Allow avatar editing via cropping, rotation and zooming - Reported by: @parasharrajat

[isagoico]

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!

---

Action Performed:

1. Navigate to profile settings
2. Upload a new profile picture

Expected Result:

User should be able to edit the picture before setting the new avatar (Like zooming and cropping)

Actual Result:

User is not given the option to edit the profile picture when uploading a new avatar.

Workaround:

User has to edit the picture before uplaoding.

Platform:

Where is this issue occurring?

• Web
• iOS
• Android
• Desktop App
• Mobile Web

Version Number: 1.1.14-0

Reproducible in staging?: Yes
Reproducible in production?: Yes

Logs: https://stackoverflow.com/c/expensify/questions/4856

Notes/Photos/Videos: Any additional supporting documentation

Expensify/Expensify Issue URL:

Issue reported by: @parasharrajat
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1636994572206400

View all open jobs on GitHub

Solution

This is the final UI agreed upon:

The two new icons you will need (zoom, rotate) are here: Icons.zip

[MelvinBot]

Triggered auto assignment to @mallenexpensify (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

[MelvinBot]

Triggered auto assignment to @deetergp (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

[mallenexpensify]

Added Engineering label , assigned Deeter, to check to ensure this can be worked on by a contributor

[deetergp]

I can't think of any reason this couldn't be done by a contributor.

[MelvinBot]

Triggered auto assignment to @tgolen (Exported), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

[mallenexpensify]

https://www.upwork.com/jobs/~01c68ed933432e239b
Job posted 😎

[parasharrajat]

Looks like this needs Design @Expensify/design

[mallenexpensify]

I think you're right @parasharrajat. Let me assign someone for 👀 and review to see what their initial thoughts, pointers, feedback are. @shawnborton (who was auto-assigned) can you take a look?

[MelvinBot]

Triggered auto assignment to @shawnborton (Design), see these Stack Overflow questions for more details.

[tgolen]

I would like to push the pause button on this for a bit. I think it's an interesting feature request, but I think we should spend a little more time determining if it's a feature that we WANT right now or not. I would like to see whoever proposed it (I think @parasharrajat) craft a problem solution statement for this. I am not sure what problem it solves, so therefore, I am not yet on board with this "solution".

[tgolen]

(let me know if you're familiar with how we do problem/solution statements or not)

[parasharrajat]

I definitely don't know your problem solution technique 😀 but I can try to create a problem statement.

[tgolen]

Haha, sure! So this is the template that we use for our internal issue repo:

# Problem

# Why is this important?

# Solution

Some suggestions to help you out:

• Good problems don't ever mention the solution
• Framing things from the customer perspective is most helpful
• A good forumla to follow for the problem is "When X happens, it leads to Y, which is important to solve because of Z"

[tgolen]

not overdue

[vitalii-tb]

Not overdue

[tgolen]

Still not overdue

[tgolen]

I'm going to change the label so this doesn't keep going overdue.

[mallenexpensify]

I unassigned Ariel and assigned myself (I was OOO for a while)

What are we waiting for to close this issue and to pay the contributors? This is the associated PR, correct? Looks like it merged a while ago

[parasharrajat]

Yeah, waiting for the deployment cycle to complete.

[parasharrajat]

This should be ready for payment now.

[mallenexpensify]

@tgolen , can you chime in about releasing payment plz. Looks like the PR hit staging but never production, so I'm a bit confused. Also, the title never auto-updated.
#8511 (comment)

[tgolen]

I know the comments were broken for one deploy cycle, maybe that's what got messed up here. I just checked production and the feature is definitely there, so I think we are fine to move forward with payment.

[parasharrajat]

PR was deployed to PROD 14 days back #10253 (comment)

[mallenexpensify]

Thanks @tgolen

Paid @tabakharniuk $8,000! Thanks!
@parasharrajat , your hire/job got cancelled somehow. Can you please accept the new job I created?
https://www.upwork.com/jobs/~0187ef9e300849eeba

Please post here once you have (and... remind me to add $250 for reporting :) )

[vitalii-tb]

Thank you @parasharrajat @mallenexpensify @tgolen!

[mallenexpensify]

Paid @parasharrajat $8250 for C+ and reporting. Thanks again for the help, happy to see this big feature request wrapped up.

Curious of any learnings from this one that might help with other feature requests (since... bugs are much more straight forward). I think there needs to be a significant initial investment in the design and concrete deliverable in order to alleviate excessive back n' forth. Espec with contributors and C+ who won't have as much context and understanding of how the feature will work on/with our product and roadmap. Anywho... gonna close this now :)

[mallenexpensify]

@tabakharniuk , I just used the feature and it worked like a charm! Thanks again for the help