-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
MM-54199 - attachment author name missing on mobile #7575
Conversation
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.
Great!
app/components/post_list/post/body/content/message_attachments/attachment_author.test.tsx
Show resolved
Hide resolved
@DHaussermann friendly reminder on this one :) . Thanks! |
/update-branch |
Building app in separate branch. |
@pvev maybe I've misunderstood something here but, I don't see a change. The authors name is still not visible in the post attachment data when I view it with the mobile client. I also used the Matterpoll example. I tested this on Android (iOS testing is blocked for me for reasons unrelated to this PR) as follows:
I using a Samsung Galaxy S22 running Android 13 with the stock One UI 5.1 launcher. |
Sorry @pvev, we can ignore the comment above ☝️ After repeating the steps and side-loading the app again I can see your change 🎉 Testing is ongoing - I'm still trying to get my iOS device working with test builds. I will post an update on this shortly. |
Tested and passed.
LGTM! Thanks @pvev for helping troubleshoot my deployment issue on this one 👍 |
Summary
There was a small display issue with the author name in the matterpoll attachment in mobile view posts. This PR makes sure to display the author name either the icon is provided or not.
Ticket Link
https://mattermost.atlassian.net/browse/MM-54199
Checklist
Device Information
This PR was tested on:
Screenshots
Before:
After:
Release Note