Differential D25626
The feed "created this task" should be the first one Authored by valerio.bozzolan on May 8 2024, 10:08. Tags None Referenced Files
Subscribers Tokens
Details
Create a Maniphest task, Resolved, High priority. Before this change, this was the feed in the object page:
After this change, this is the feed:
This also improves the situation in the /feed/ page. So you see the creation now. Ref T15816 Create an object in any mentioned application below, setting all fields. Then, check the history of the object itself. Check also the /feed/ page. This change improves the situation in:
This change has no impact on these that seem already OK:
This change has no impact on these that seem still in the wrong order:
The above situation in Differential is noted, but it's unrelated,
Diff Detail
Event TimelineComment Actions I can confirm this works as expected for the task view itself in e.g. http://phorge.localhost/T1168 . This seems to also fix http://phorge.localhost/feed/ so it does not only show me about setting high priority on a new task, but instead shows that the task was created? That would be a nice side effect too IMO. I'm a little bit wondering if you could elaborate how you came to the value 140? It might make total sense, just wondering what you compared with. Comment Actions Uh! I can confirm that. I'm adding that in commit message.
Thanks. For details about that 140, see comments in line 34 and 35, and super-nice details in T15816 Comment Actions I currently cannot think of other potential side effects (mail notifications about task creation seem to have had a [Created] prefix taking priority over other actions for years, and web notifications in the top bar also still behave as before), so I'll give my +1 |