-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
[🐴] show deleted account for missing.invalid
#4014
Conversation
haileyok
commented
May 14, 2024
•
edited
Loading
edited
Your Render PR Server URL is https://social-app-pr-4014.onrender.com. Follow its progress at https://dashboard.render.com/web/srv-cp1qgpicn0vc73c3gvng. |
|
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.
Minor display name logic problem
const displayName = isDeletedAccount | ||
? 'Deleted Account' | ||
: otherUser?.displayName ?? otherUser?.handle |
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.
Should be
const displayName = isDeletedAccount
? 'Deleted Account'
: otherUser?.displayName || otherUser?.handle
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.
How do you get it to do the "accept suggested changes" button?
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.
Why bluesky why
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.
le sigh
* origin/main: (48 commits) [🐴] Show if user can be messaged in new chat search (#4021) Make more localizable (#3962) [🐴] change `Follows Only` to `People I Follow` (#4028) Reset after 5 min (#4026) [🐴] Block states, read only (#4022) [🐴]Add DM push notification sound (#4000) remove serviceurl gate (#4019) reword subtitle (#4017) [🐴] show deleted account for `missing.invalid` (#4014) [🐴] use "Chats" instead of "Messages" (#4013) 100vh settings screen (#4015) Run intl extract (#4016) [🐴] Settings screen (#3830) [🐴] Message drafts (#3993) Update Korean localization (#3887) Update catalan messages.po (#3984) Fix dep (#4011) [🐴] Make status checks easier, fix load state (#4010) [🐴] Chat muted state (#3988) [🐴] Swap in new package, update usages (#3992) ...