-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Threads - Nothing happens when Clicking Profile picture in detail of Group Chat thread #39760
Comments
Triggered auto assignment to @greg-schroeder ( |
We think this issue might be related to the #vip-vsb. |
ProposalPlease re-state the problem that we are trying to solve in this issue.Threads - Nothing happens when Clicking Profile picture in detail of Group Chat thread What is the root cause of that problem?Here we prevent navigation to What changes do you think we should make in order to solve the problem?Remove this condition or update the check to What alternative solutions did you explore? (Optional)Add prop |
This feature is still in development and the decision to return early was done intentionally c.c. @marcaaron |
ProposalPlease re-state the problem that we are trying to solve in this issue.Threads - Nothing happens when Clicking Profile picture in detail of Group Chat What is the root cause of that problem?The main problem with issue is that we have removed the ability to open an icon if it is a group chat App/src/pages/ReportDetailsPage.tsx Lines 218 to 222 in 0e6dba4
What changes do you think we should make in order to solve the problem?To fix this issue we can update this condition and add ability to press when it's group chat thread
What alternative solutions did you explore? (Optional)As alternative we can change cursor when it's group chat
App/src/components/RoomHeaderAvatars.tsx Line 45 in 0e6dba4
|
This might be expected - can you confirm this comment #39760 (comment) @marcaaron? <3 |
Yes, not a bug we just didn't add it yet. We should add it. But let's get the feature mostly done first. |
Current assignee @greg-schroeder is eligible for the NewFeature assigner, not assigning anyone new. |
Okay this is held right now, will check for updates weekly on group chat progress |
Held |
Held |
@marcaaron do you know if "Front End PR 2" is done yet? |
Oh sorry, yes, it's been done for quite a while 🥲 |
We actually have an open issue for this already here someone is working on it. Gonna close this one out. |
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Version Number: v1.4.60-12
Reproducible in staging?: Y
Reproducible in production?: Y
Email or phone of affected tester (no customers): [email protected]
Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team
Action Performed:
Expected Result:
If there is a pointer cursor. Clicking the profile must display the picture in full size.
Actual Result:
Clicking the profile does nothing even if there is a Pointer cursor
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Bug6437755_1712223237696.Thread_Profile_Picture.mp4
View all open jobs on GitHub
The text was updated successfully, but these errors were encountered: