Page MenuHomePhorge
Feed All Stories

Feb 28 2024

aklapper closed D25541: Fix typo "pohrge" in "Adding New Classes" documentation.
Feb 28 2024, 12:52
aklapper committed rP74ec4f7882a3: Fix typo "pohrge" in "Adding New Classes" documentation.
Fix typo "pohrge" in "Adding New Classes" documentation
Feb 28 2024, 12:52
valerio.bozzolan added inline comments to D25475: Drag & Drop Task Cover Image: also attach.
Feb 28 2024, 12:46
valerio.bozzolan requested changes to D25505: Improve handling queries with "Current Viewer" set while not logged in.

Let's visually mark this to attract people.

Feb 28 2024, 12:31
valerio.bozzolan accepted D25542: Replace all phurl.io short URIs with target URIs.

I've carefully read the diff :) Thanks

Feb 28 2024, 12:28
valerio.bozzolan added a comment to T15121: Contributor Agreement.

Totally agree. Thanks for any edit in the stub.

Feb 28 2024, 12:25 · Phorge
valerio.bozzolan updated the task description for T15746: https://phurl.io is a HTTP 503 error.
Feb 28 2024, 12:23
valerio.bozzolan accepted D25543: Replace all phurl.io short URIs with target URIs.

Have you digged Internet Archive to fix this? :)

Feb 28 2024, 12:17
valerio.bozzolan updated the task description for T15746: https://phurl.io is a HTTP 503 error.
Feb 28 2024, 12:16
aklapper added a comment to T15121: Contributor Agreement.

I don't think that the author should ever allow to re-license its work. [...] Normal volunteers do not accept that.

Feb 28 2024, 12:16 · Phorge
valerio.bozzolan accepted D25541: Fix typo "pohrge" in "Adding New Classes" documentation.

Go pohrge!

Feb 28 2024, 11:55
aklapper updated the diff for D25543: Replace all phurl.io short URIs with target URIs.

Clean up my local testing mess

Feb 28 2024, 11:39
aklapper added a comment to T15746: https://phurl.io is a HTTP 503 error.

Most phurl.io short URIs point to old upstream tasks. The two related patches make the code at least point directly to these tasks (to still be found on https://web.archive.org/ if secure.phabricator.com ever went down) as there's no direct content equivalent on phorge.it.

Feb 28 2024, 11:34
aklapper requested review of D25543: Replace all phurl.io short URIs with target URIs.
Feb 28 2024, 11:31
aklapper added a revision to T15746: https://phurl.io is a HTTP 503 error: D25543: Replace all phurl.io short URIs with target URIs.
Feb 28 2024, 11:31
aklapper requested review of D25542: Replace all phurl.io short URIs with target URIs.
Feb 28 2024, 11:29
aklapper added a revision to T15746: https://phurl.io is a HTTP 503 error: D25542: Replace all phurl.io short URIs with target URIs.
Feb 28 2024, 11:29
aklapper added a comment to T15745: Cannot install extension "Diagrams" (Failed to load symbol "DiagramTransactionType").

@valerio.bozzolan Are there any prehistoric traces that this extension has been successfully used in the past? :P

Feb 28 2024, 11:08 · Diagrams (Extension)
littleggghost added a comment to Q114: Subtypes deadline switch to colored calendar icons (Answer 146).

I tried with only merge this commit https://phabricator.wikimedia.org/rPHABae334e4589a0413c1eb8a9e8d993e553bfeff478
but still not work.

Feb 28 2024, 10:41
avivey added a comment to T15746: https://phurl.io is a HTTP 503 error.

It's a Phacility server, managed at https://secure.phabricator.com/phurl/

Feb 28 2024, 10:14
aklapper created T15746: https://phurl.io is a HTTP 503 error.
Feb 28 2024, 08:42
20after4 added a comment to T15675: Visually indicate new user accounts.

I agree, badges seems like the right way to go. They are an underutilized feature really, IMO.

Feb 28 2024, 08:37 · Feature Requests
20after4 updated the answer details for Q114: Subtypes deadline switch to colored calendar icons (Answer 146).
Feb 28 2024, 08:35
aklapper requested review of D25541: Fix typo "pohrge" in "Adding New Classes" documentation.
Feb 28 2024, 08:34
20after4 updated the answer details for Q114: Subtypes deadline switch to colored calendar icons (Answer 146).
Feb 28 2024, 08:33
20after4 added a comment to Q114: Subtypes deadline switch to colored calendar icons (Answer 146).

@littleggghost So it turns out that in order to enable this feature, there was a patch to Phabricator core. That patch isn't part of the wikimedia extension, and hasn't been merged into phorge.

Feb 28 2024, 08:31
avivey added a comment to T15675: Visually indicate new user accounts.

I've looked down some of the links of this request, and it looks like the definition of "new user" should be install-specific, and open to interpretation.

Feb 28 2024, 08:22 · Feature Requests
littleggghost added a comment to Q114: Subtypes deadline switch to colored calendar icons (Answer 146).

When I search in

resources/celerity/map.php
Feb 28 2024, 07:59
littleggghost added a comment to Q114: Subtypes deadline switch to colored calendar icons (Answer 146).

I run this cmd in this docker container, with output:

root@f0d9837493f0:/var/www/phorge/phorge# ./bin/celerity map
Rebuilding 1 resource source(s).
Rebuilding resource source "phabricator" (CelerityPhabricatorResources)...
Found 113 binary resources.
Found 399 text resources.
Found 11 packages.
Writing map "resources/celerity/map.php".
Done.
Feb 28 2024, 07:41
aklapper abandoned D25477: Visually indicate new user accounts.

Makes sense, thanks everyone. Let's decline for the time being.

Feb 28 2024, 07:03

Feb 27 2024

20after4 updated subscribers of Q114: Subtypes deadline switch to colored calendar icons (Answer 146).

@littleggghost one thing you might want to try is updating the celerity map. You may be missing the custom css rules to style the deadline appropriately. I’m not familiar enough with the phorge docker image to be able to tell you exactly how that is accomplished. With non-containerized phorge instances you would just run bin/celerity map from within the phorge root directory.

Feb 27 2024, 22:11
20after4 added Q114: Subtypes deadline switch to colored calendar icons (Answer 146).
Feb 27 2024, 21:53
littleggghost asked Q114: Subtypes deadline switch to colored calendar icons.
Feb 27 2024, 13:14
avivey renamed T15512: Make Flags more useful from Reimplement "Flags" to be based on Edges to Make Flags more useful.
Feb 27 2024, 12:21
avivey claimed T15512: Make Flags more useful.

I changed my mind again. No need for edges (short story is, getting the "color" property from the edge is annoying).

Feb 27 2024, 12:19
valerio.bozzolan created T15745: Cannot install extension "Diagrams" (Failed to load symbol "DiagramTransactionType").
Feb 27 2024, 10:52 · Diagrams (Extension)
20after4 added a comment to T15743: Support for math (LaTeX) in Remarkup.
Feb 27 2024, 01:21 · Feature Requests, Remarkup

Feb 26 2024

20after4 added a comment to T15743: Support for math (LaTeX) in Remarkup.

Interestingly, I just discovered Wikimedia has a pretty cool project to render math on the server side, by transpiling some js into php:

Feb 26 2024, 19:18 · Feature Requests, Remarkup
20after4 awarded T15718: Guide/demo on "How To Create Your First Extension" a Orange Medal token.
Feb 26 2024, 18:49 · Extension Development, Phactory: Community Projects
20after4 updated the answer details for Q113: How to create a bug report (Answer 145).
Feb 26 2024, 18:48
20after4 added Q113: How to create a bug report (Answer 145).
Feb 26 2024, 18:47
20after4 added a comment to D25540: Add PhutilRemarkupHexColorCodeRule, a new remarkup rule to format color codes.

I went down a rabbit hole while trying to add unit tests for this. Our core remarkup unit tests are pretty thorough but they only seem to cover the "core" remarkup engine and syntax. When I added a new test case for this it would pass but always caused a different test case to fail in weird ways. I haven't yet figured out what part of the unit test or engine is stateful such that my unit test alters the state in a way that breaks a later test. If I change the order that tests run in, it just causes another different test case to fail.

Feb 26 2024, 18:40
20after4 updated the diff for D25540: Add PhutilRemarkupHexColorCodeRule, a new remarkup rule to format color codes.

Fix lint again

Feb 26 2024, 18:36
20after4 updated the diff for D25540: Add PhutilRemarkupHexColorCodeRule, a new remarkup rule to format color codes.

fix lint.

Feb 26 2024, 18:35
20after4 updated the diff for D25540: Add PhutilRemarkupHexColorCodeRule, a new remarkup rule to format color codes.

Added contrasting color calculation

Feb 26 2024, 18:33
jpeezious asked Q113: How to create a bug report.
Feb 26 2024, 18:02
20after4 added a comment to T15743: Support for math (LaTeX) in Remarkup.

There is also MathJax, a subset of LaTex that only covers the mathematical notation without the page layout functionality.

Feb 26 2024, 17:42 · Feature Requests, Remarkup
valerio.bozzolan added a comment to T15121: Contributor Agreement.

Unrelated note, but interestingly the contributor agreement chooser (https://contributoragreements.org/) is broken. I've already sent an email to their support.

Feb 26 2024, 11:09 · Phorge

Feb 25 2024

valerio.bozzolan added a comment to T15743: Support for math (LaTeX) in Remarkup.

I see that it depends on a NodeJS external script (server-side).

Feb 25 2024, 15:34 · Feature Requests, Remarkup
valerio.bozzolan added a comment to D25540: Add PhutilRemarkupHexColorCodeRule, a new remarkup rule to format color codes.

Interesting, I agree. Note that also #bbb is valid but I see that 6 is minimum characters.

Feb 25 2024, 11:58
aklapper added a comment to D25540: Add PhutilRemarkupHexColorCodeRule, a new remarkup rule to format color codes.

Interesting. What's the use case you have in mind?

Feb 25 2024, 10:15
avivey awarded T15744: Render html color codes in remarkup a Grey Medal token.
Feb 25 2024, 08:59 · Discussion Needed, Remarkup
avivey awarded T15743: Support for math (LaTeX) in Remarkup a Yellow Medal token.
Feb 25 2024, 08:18 · Feature Requests, Remarkup
20after4 claimed T15744: Render html color codes in remarkup.
Feb 25 2024, 04:14 · Discussion Needed, Remarkup
20after4 triaged T15744: Render html color codes in remarkup as Normal priority.
Feb 25 2024, 04:14 · Discussion Needed, Remarkup
20after4 planned changes to D25540: Add PhutilRemarkupHexColorCodeRule, a new remarkup rule to format color codes.

This is not yet tested.

Feb 25 2024, 04:13
20after4 updated the diff for D25540: Add PhutilRemarkupHexColorCodeRule, a new remarkup rule to format color codes.

fix lint error.

Feb 25 2024, 04:12
20after4 requested review of D25540: Add PhutilRemarkupHexColorCodeRule, a new remarkup rule to format color codes.
Feb 25 2024, 04:10
20after4 added a revision to T15744: Render html color codes in remarkup: D25540: Add PhutilRemarkupHexColorCodeRule, a new remarkup rule to format color codes.
Feb 25 2024, 04:10 · Discussion Needed, Remarkup
20after4 added a comment to T15744: Render html color codes in remarkup.

This would be useful for discussions about design, specifically I wished for this feature while reviewing D25491: Improve contrast of Links in Dark Mode

Feb 25 2024, 00:53 · Discussion Needed, Remarkup
20after4 created T15744: Render html color codes in remarkup.
Feb 25 2024, 00:48 · Discussion Needed, Remarkup
20after4 created T15743: Support for math (LaTeX) in Remarkup.
Feb 25 2024, 00:40 · Feature Requests, Remarkup

Feb 24 2024

l2dy awarded D25537: Update PHP versions in install guide: Remove 5, Add 8. a Cup of Joe token.
Feb 24 2024, 12:54
avivey removed 1 auditor(s) for rPaf06bfb1cc49: Make Changeset ID for render cache explicit: Wikivu.
Feb 24 2024, 10:03
avivey added a comment to D25079: Trigger: Add Sound "Coin".

Yeah, I don't like the diviner page.
For other included 3rd parties, we often put a license file right next to the asset:

Feb 24 2024, 09:46

Feb 23 2024

Iniquity added a comment to D25491: Improve contrast of Links in Dark Mode.

Hello! Are you sure changing the links color is a good idea? Perhaps we should just make the background darker?

I actually think a darker background and lighter link color would be good. Maybe not quite as light as #accadf, perhaps #46B2FF or so

Feb 23 2024, 21:43 · UX, Accessibility
Wikivu added a comment to rP1ddb953b9f77: Update Diviner documentation to reference Phorge instead of Phabricator..

Volkan 🌋 ural

Feb 23 2024, 17:30
aklapper added a comment to D25079: Trigger: Add Sound "Coin".
Feb 23 2024, 15:16
avivey closed D25539: Fix an instance of `strlen(null)`.
Feb 23 2024, 15:06
avivey committed rP8667ed45fada: Fix an instance of `strlen(null)`.
Fix an instance of `strlen(null)`
Feb 23 2024, 15:06
valerio.bozzolan updated subscribers of D25079: Trigger: Add Sound "Coin".

I'm also unsure and I would agree if somebody suggests to omit the Diviner page. Relevant discussion also from @avivey

Feb 23 2024, 15:06
aklapper added a comment to D25079: Trigger: Add Sound "Coin".

I guess I'm fine with adding that mp3 file.
I am personally not convinced by the creation of src/docs/flavor/project_credits.diviner: I've seen way too many "let's create some list, with best intentions, and at some point unfortunately forget about updating it" examples in FOSS projects. Every added line in software projects creates additional maintenance costs (and in some cases bitrot); human resources are in theory unlimited and in reality rather sparse.
Credits could go into the git commit message, maybe?

Feb 23 2024, 14:52
valerio.bozzolan added a comment to D25079: Trigger: Add Sound "Coin".

It's the anniversary of this super-important killer feature 💌

Feb 23 2024, 14:28
Ekubischta renamed T15742: When creating a Diffusion Repo, add helpful text to each field from When creating a Diffusion Rep, add helpful text to each field to When creating a Diffusion Repo, add helpful text to each field.
Feb 23 2024, 14:19 · UX, Diffusion
Ekubischta created T15742: When creating a Diffusion Repo, add helpful text to each field.
Feb 23 2024, 14:19 · UX, Diffusion
valerio.bozzolan accepted D25539: Fix an instance of `strlen(null)`.
Feb 23 2024, 13:14
avivey closed D25537: Update PHP versions in install guide: Remove 5, Add 8..
Feb 23 2024, 11:35
avivey committed rP7d8c84a7bdc8: Update PHP versions in install guide: Remove 5, Add 8..
Update PHP versions in install guide: Remove 5, Add 8.
Feb 23 2024, 11:35
avivey added inline comments to D25539: Fix an instance of `strlen(null)`.
Feb 23 2024, 11:32
avivey added Q112: Deprecation Notice from PhrictionRemarkupRule.php:194 (Answer 144).
Feb 23 2024, 11:30
avivey requested review of D25539: Fix an instance of `strlen(null)`.
Feb 23 2024, 11:29
avivey added a revision to T15064: Make Phorge compatible with PHP 8.1/8.2/8.3/8.4: D25539: Fix an instance of `strlen(null)`.
Feb 23 2024, 11:29 · PHP 8 support
avivey awarded Q112: Deprecation Notice from PhrictionRemarkupRule.php:194 a Like token.
Feb 23 2024, 10:51 · Deprecation Notice
bekay asked Q112: Deprecation Notice from PhrictionRemarkupRule.php:194.
Feb 23 2024, 10:11 · Deprecation Notice
20after4 added a comment to T15719: Phorge mail may not meet Google's email sender guidelines.
In T15719#15304, @Tgr wrote:

Given how it's displayed on the Gmail UI, IMO unsubscribing from all emails or at least all emails of the given type (e.g. all Maniphest emails if it's a Maniphest notification) is the only thing that makes sense. If I mark an email as spam, and then use the unsubscribe option instead, I would not expect to continue getting identical email from all the other tasks.
No idea how (or if) other email clients handle the header, though.

Feb 23 2024, 08:54 · Maniphest, Mail

Feb 22 2024

avivey added a comment to T15512: Make Flags more useful.

This was both harder and easier then I expected. I got to crash the IDE and hard-code a bunch of stuff, but this actually works. Didn't even start on the Edges implementation yet - I'm directly using the Flags table.

Feb 22 2024, 22:28
aklapper added a comment to T15738: "parent"/"child" relationship for "Related Revisions" is different to "Related Tasks".

See discussion and reasons in https://secure.phabricator.com/T11833

Feb 22 2024, 13:21 · UX, Differential
aklapper renamed T15738: "parent"/"child" relationship for "Related Revisions" is different to "Related Tasks" from Make "Related Revisions" more similar to "Related Tasks" (magical quiz inside) to "parent"/"child" relationship for "Related Revisions" is different to "Related Tasks".
Feb 22 2024, 13:21 · UX, Differential
CryingWolf awarded T15048: Allow awarding Tokens to individual Comments a Mountain of Wealth token.
Feb 22 2024, 10:32 · Comments
20after4 added a comment to D25491: Improve contrast of Links in Dark Mode.

Hello! Are you sure changing the links color is a good idea? Perhaps we should just make the background darker?

Feb 22 2024, 09:58 · UX, Accessibility
20after4 added a comment to T15048: Allow awarding Tokens to individual Comments.

The only progress I've made is to think through the implementation but I haven't actually started coding on it. It is sort of high on my wish list so I may get to it soon.

Feb 22 2024, 09:50 · Comments

Feb 21 2024

bekay added a comment to T15741: Diffusion pattern search not working as expected.

The word solution would solve it from the expecting side, you are right. Here in my company the search is used frequently and only as a plain text search. So we have to escape the special regex signs time and again. Just would be interested if this is seen as an issue by others who uses the feature.

Feb 21 2024, 12:48 · Diffusion
avivey added a comment to T15741: Diffusion pattern search not working as expected.

We can probably add a checkbox, and some nicer handling for the exception. Or possibly, just adding the word "regex" might solve this from the "user expectation" side.

Feb 21 2024, 12:43 · Diffusion
bekay added a comment to T15741: Diffusion pattern search not working as expected.

Thanks for the clarification. I still think we should tackle the exception.

Feb 21 2024, 12:41 · Diffusion
avivey added a comment to T15741: Diffusion pattern search not working as expected.

The word "pattern" was meant to imply "regex pattern".
Specifically - the ability to search for a regex is much more useful then a plain-text only search (and costs us basically nothing).

Feb 21 2024, 12:35 · Diffusion
sicenul closed Q111: How to disable noreply mailbox? as resolved.
Feb 21 2024, 11:42
sicenul added a comment to Q111: How to disable noreply mailbox? (Answer 143).

Thanks, it works

Feb 21 2024, 11:42
bekay updated the task description for T15741: Diffusion pattern search not working as expected.
Feb 21 2024, 10:52 · Diffusion
bekay created T15741: Diffusion pattern search not working as expected.
Feb 21 2024, 10:51 · Diffusion
bekay created T15740: Crumb menu wrapping is deficient.
Feb 21 2024, 10:35 · UX