Fix regression: Verifyable link to new account after profile moved #21844
Conversation
| @@ -23,7 +23,7 @@ export default class MovedNote extends ImmutablePureComponent { | |||
| </div> | |||
|
|
|||
| <div className='moved-account-banner__action'> | |||
| <Link to={`/@${to.get('acct')}`} className='detailed-status__display-name'> | |||
| <Link rel='me' href={to.get('url')} to={`/@${to.get('acct')}`} className='detailed-status__display-name'> | |||
There was a problem hiding this comment.
The to= property already renders a href and I doubt multiple href are allowed. Also this entire component is not rendered serverside, so the resulting HTML contains neither the link nor your fixed version of it. I don't think this will work.
There was a problem hiding this comment.
you likely want to patch this file instead (or possibly in addition): https://github.com/mastodon/mastodon/blob/main/app/views/accounts/show.html.haml#L8-L9
|
Hi! Have you verified this would fix your issue? I don't think that's related to the Your proposal would have an appropriate |
|
i haven't verified anything and i'm not too familiar with the mastodon codebase. maybe someone with a better understanding of the codebase could ensure a verifyable link to a transferred profile could be included in the delivered html of a profile page if an account is transferred. |
See issue #21845