In D25626#17531, @aklapper wrote: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.
- Queries
- All Stories
- Search
- Advanced Search
- Transactions
- Transaction Logs
Feed All Stories
All Stories
All Stories
May 8 2024
May 8 2024
Maybe relevant, from my Postfix logs:
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.
Tested this patch on http://phorge.localhost/feed/transactions/ and in the feed at the bottom of http://phorge.localhost/W20 itself.
Also tested changing panel visibility from Public to Admin only and accessing these URIs as an average user.
All working as expected, no explosions.
valerio.bozzolan updated the summary of D25625: Dashboard Text Panel: improve feed when changing Description.
valerio.bozzolan updated the summary of D25625: Dashboard Text Panel: improve feed when changing Description.
valerio.bozzolan requested review of D25625: Dashboard Text Panel: improve feed when changing Description.
valerio.bozzolan renamed T15816: The feed "created this task" should be the first one from The feed "created this task" should be the first one, not the last one to The feed "created this task" should be the first one.
valerio.bozzolan triaged T15817: Dashboard Text Panel: Incomplete Transaction "text.text" when changing Description as Wishlist priority.
valerio.bozzolan changed the visibility for F2182844: Phorge Text Panel with text-text incomplete transaction.png.
valerio.bozzolan updated the task description for T15816: The feed "created this task" should be the first one.
valerio.bozzolan added a comment to T15815: Make i18n string extraction ignore strings in test cases.
Thanks. Completely agree.
In T15815#17250, @valerio.bozzolan wrote:being able to have people running unit tests and see the results in their language was a feature
valerio.bozzolan updated the name of F2182823: Task history with created this task as third action.png from "image.png" to "Task history with created this task as third action.png".
valerio.bozzolan added a project to T15816: The feed "created this task" should be the first one: Bug Reports.
avivey added a comment to T15096: Discuss Arcanist as a barrier to adoption of Phorge and how to address the underlying issues..
In T15096#12185, @valerio.bozzolan wrote:Another problem dramatically frequent for newcomers (at least in my office).
It's relatively too much easy to start with a similar commit message:
...
valerio.bozzolan awarded T15815: Make i18n string extraction ignore strings in test cases a Like token.
(run the full tests, and then count it as my + O1 Accept).
I think the costs of the extra translations are low, but the cost of "the tests are not being translated for developers" are even lower.
I'd be worried that this might break some tests that do relate to translation/extraction, but running arc unit --everything should solve that.
valerio.bozzolan added a comment to D25624: Make i18n string extraction ignore strings in test case files.
Thanks. To reviewers: maybe relevant, maybe not:
valerio.bozzolan added a comment to T15815: Make i18n string extraction ignore strings in test cases.
I 100% see the point of avoiding to "waste translators' time", but I also see that completely removing tests is a big choice for upstream: being able to have people running unit tests and see the results in their language was a feature, so we also "waste a feature".
I tested this one-liner as part of the downstream conversation that pppery and I had in https://phabricator.wikimedia.org/T363364, still I would ideally prefer another upstream opinion before accepting this revision as O1.
May 7 2024
May 7 2024
Re-diff with linting and tests working hopefully
valerio.bozzolan updated the diff for D25621: Show login page if a search token requires a valid viewer.
fix "abstract" function name
valerio.bozzolan published D25621: Show login page if a search token requires a valid viewer for review.
Fix exception trying to rename user to their previous username
docs/file_domain: Update cloudflare spelling and URLs
Thanks. Confirmed reading this:
Nice catch :) Can reproduce. Tested.
cover all known usages, but set as stub
valerio.bozzolan added a comment to T15096: Discuss Arcanist as a barrier to adoption of Phorge and how to address the underlying issues..
At least, please go to GitLab, instead of GitHub, so at least you use Free Software :)
Something I stumbled upon few months ago (via forgejo): agit-flow[^1]. Maybe worth sharing here -- for teh record? It does have some CLI client (git-repo) but it's optional and push via git push origin HEAD:refs/for/<target-branch>/<session> would trigger the Code Review process, without gerrit's Change-Id hack.
micax added a comment to T15096: Discuss Arcanist as a barrier to adoption of Phorge and how to address the underlying issues..
Just some (probably final) thoughts on arcanist and phabricator/phorge. After quite a bit of discussion internally, we are moving on to GitHub with our repositories (which were previously stored in another Git server - not Phabricator). But that move also spells the end for our/my 10+ year long use of Phabricator, because when we move to GitHub, we'll also be ditching Phabricator for code reviews.
May 6 2024
May 6 2024
valerio.bozzolan added a parent task for T15814: Files: reduce number of orphan transformed files: T15768: Better tracking tools for Files.
valerio.bozzolan triaged T15814: Files: reduce number of orphan transformed files as Normal priority.
valerio.bozzolan added a comment to Q136: [BUG] Arcanist exception during the initialization on PHP 8.1 (Answer 168).
lol
valerio.bozzolan awarded Q136: [BUG] Arcanist exception during the initialization on PHP 8.1 a Grey Medal token.
zhe added a comment to Q136: [BUG] Arcanist exception during the initialization on PHP 8.1 (Answer 168).
After upgrading PHP from 8.1.22 to 8.1.27, problem fixed. Thank you again, bozz.
zhe added a comment to Q136: [BUG] Arcanist exception during the initialization on PHP 8.1 (Answer 168).
Thank you, Bozz.
valerio.bozzolan closed Q136: [BUG] Arcanist exception during the initialization on PHP 8.1 as resolved.
valerio.bozzolan added a comment to Q136: [BUG] Arcanist exception during the initialization on PHP 8.1.
valerio.bozzolan added a comment to Q136: [BUG] Arcanist exception during the initialization on PHP 8.1.
valerio.bozzolan added a comment to D25505: Improve handling queries with "Current Viewer" set while not logged in.
Thanks again for this patch that highlighted the valley and the mountain in a very clear way.
May 5 2024
May 5 2024
valerio.bozzolan triaged T15704: Cryptic exception when listing tasks "Closed By: Current Viewer" while logged out as Normal priority.
Facts: Fix displaying charts for anonymous users
In D25608#17388, @avivey wrote:This is probably the right fix (or maybe moving shouldAllowPublic() to the base PhabricatorFactController).
This is probably the right fix (or maybe moving shouldAllowPublic() to the base PhabricatorFactController).
valerio.bozzolan updated the task description for T15407: People: profile picture should be editable by their author (not by "No one").
valerio.bozzolan updated the name of F2179225: Custom Edit Permission when the Moon is Full.png from "Screenshot_20240505_160718.png" to "Custom Edit Permission when the Moon is Full.png".
valerio.bozzolan claimed T15407: People: profile picture should be editable by their author (not by "No one").
After a small database inspection, it seems that uploading the picture causes:
valerio.bozzolan updated the task description for T15407: People: profile picture should be editable by their author (not by "No one").
valerio.bozzolan added a comment to D25505: Improve handling queries with "Current Viewer" set while not logged in.
I'm currently stuck on this since I'm trying to refactor the functions to support an indications about "I work for everybody" and "I work for only logged-in users" to be able to distinguish the case "The token exists" VS "The token exists but it's not visible" VS "The token does not exist"
waldyrious updated the task description for T15270: "Access Denied: Restricted File" strings concatenated without whitespace.
valerio.bozzolan updated the task description for T15270: "Access Denied: Restricted File" strings concatenated without whitespace.
valerio.bozzolan committed rPc8fb5046909c: Fix the whitespace issue in the message "Access Denied: Restricted File" (authored by waldyrious).
Fix the whitespace issue in the message "Access Denied: Restricted File"
valerio.bozzolan updated the summary of D25419: Fix the whitespace issue in the message "Access Denied: Restricted File".
valerio.bozzolan accepted D25419: Fix the whitespace issue in the message "Access Denied: Restricted File".
Thanks to @waldyrious for the change. Tested. It works.
Content licensed under Creative Commons Attribution-ShareAlike 4.0 (CC-BY-SA) unless otherwise noted; code licensed under Apache 2.0 or other open source licenses. · CC BY-SA 4.0 · Apache 2.0