Change Details

In the Feed if there is a commit, its message can be mentioned twice: once in the title, once in the body. Probably, it would be better to avoid to repeat the same phrase twice. {F313290,size=full} At the moment the content of the feed is generated by the Audit application, from: ``` PhabricatorAuditTransaction::getBodyForFeed() ``` https://we.phorge.it/source/phorge/browse/master/src/applications/audit/storage/PhabricatorAuditTransaction.php;a25af8ccef69de5978b0904866b028d96732202b$417-424 Proposals - only for the web feed: == Proposal 1: clean the body == | Current | Proposal | |---------------------|---------------------| | {F313288,size=full} | {F313289,size=full} | == Proposal 2: clean the title == | Current | Proposal | |---------------------|---------------------| | {F313288,size=full} | {F313570,size=full} |

In the Feed if there is a commit, its message can be mentioned twice: once in the title, once in the body. Probably, it would be better to avoid to repeat the same phrase twice. {F313290,size=full} At the moment the content of the feed is generated by the Audit application, from: ``` PhabricatorAuditTransaction::getBodyForFeed() ``` https://we.phorge.it/source/phorge/browse/master/src/applications/audit/storage/PhabricatorAuditTransaction.php;a25af8ccef69de5978b0904866b028d96732202b$417-424 Proposals - only for the web feed: == ✅ Proposal 1: clean the body == | Current | Proposal | |---------------------|---------------------| | {F313288,size=full} | {F313289,size=full} | Removing the duplicate body may have more sense at the moment. So there is more creative space to then add a more useful body in the future. == ⛔ Proposal 2: clean the title == | Current | Proposal | |---------------------|---------------------| | {F313288,size=full} | {F313570,size=full} | == 🤔 Proposal 3: improving the body == Adding a more useful body would be interesting. But at the moment that would be a feature. Instead now we are just trying to fix a bug about duplicated texts. Feel free to open a related task about improving the body. Probably this means improving the summary generator to be able to exclude the title and just show the next paragraph. But we have not that code right now.

In the Feed if there is a commit, its message can be mentioned twice: once in the title, once in the body. Probably, it would be better to avoid to repeat the same phrase twice. {F313290,size=full} At the moment the content of the feed is generated by the Audit application, from: ``` PhabricatorAuditTransaction::getBodyForFeed() ``` https://we.phorge.it/source/phorge/browse/master/src/applications/audit/storage/PhabricatorAuditTransaction.php;a25af8ccef69de5978b0904866b028d96732202b$417-424 Proposals - only for the web feed: == ✅ Proposal 1: clean the body == | Current | Proposal | |---------------------|---------------------| | {F313288,size=full} | {F313289,size=full} | ==Removing the duplicate body may have more sense at the moment. So there is more creative space to then add a more useful body in the future. == ⛔ Proposal 2: clean the title == | Current | Proposal | |---------------------|---------------------| | {F313288,size=full} | {F313570,size=full} | == 🤔 Proposal 3: improving the body == Adding a more useful body would be interesting. But at the moment that would be a feature. Instead now we are just trying to fix a bug about duplicated texts. Feel free to open a related task about improving the body. Probably this means improving the summary generator to be able to exclude the title and just show the next paragraph. But we have not that code right now.