The last emails I received for from Flow looked like
which is not working (they are 302 redirections to the topic pages, without color markers for unread posts) and required me to click on the web notification in order to see what was changed:
As part of this fix, this redirect should only drop the action=edit (and maybe redlink) parameters, not the rest.
This was broken by an RFC that was approved (and apparently merged) last week. There is a hook that allows us to say we 'opt in' to various parameters in the redirect.
The other half of fixing this will be adjusting Flow\UrlGenerator to ensure its now generating /wiki/* urls
I was thinking of RedirectSpecialArticleRedirectParams, which is not applicable (only applies to special pages). Also, T14619: Use article path URLs for editing, previewing skins, etc. is apparently not merged. So it's not clear what's going on here.
May have been caused by T95581: Flow board creations do not create entry in core's page and revision tables causing red links for existent topics. Apparently, action=edit on a Flow topic always causes redirecting to the view (dropping the parameters). As part of this fix, the parameters should not be dropped.
Change 217274 had a related patch set uploaded (by Matthias Mullie):
Make sure Topic:xyz pages exist() as soon as the topic is created
I'm unable to reproduce this on my end, so just to make sure:
Change 217462 had a related patch set uploaded (by Matthias Mullie):
Guarantee a correct known status for Linker::link
Change 217462 merged by jenkins-bot:
Guarantee a correct known status for Linker::link
Change 217274 abandoned by Matthias Mullie:
Make sure Topic:xyz pages exist() as soon as the topic is created
Reason:
I agree witgh YAGNI. I put up https://gerrit.wikimedia.org/r/#/c/218025/ to explain the insert order - if we ever want to change it, we can reconsider this problem, but having a fix "just in case" is pointless.
Is this task resolved / ready for sign-off now? All patches associated with it have either been merged or abandoned, so the "needs review" status is probably outdated.
Tested in betalabs: The link looks like this: href="https://en.wikipedia.beta.wmflabs.org/w/index.php?title=Topic:Sjal0evmt3cw5ugb&topic_showPostId=sjanlbo6r8r1h6qz&fromnotif=1#flow-post-sjanlbo6r8r1h6qz"
action=edit&redlink=1 are not present.
Note: For betalabs, change https to http to see the page.