Page MenuHomePhorge
Feed All Stories

Feb 25 2024

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
bekay added a comment to T15728: Repo home view and browse view share the same menu tab.

I tried to understand the situation. Thanks.

In your opinion: on the repo view (with clone buttons) is it really important to link to the other similar "reduced" view (without clone buttons)?

Feb 21 2024, 10:15 · Discussion Needed, Diffusion
valerio.bozzolan added a project to T15048: Allow awarding Tokens to individual Comments: Comments.
Feb 21 2024, 09:29 · Comments
valerio.bozzolan added a comment to T15728: Repo home view and browse view share the same menu tab.

I tried to understand the situation. Thanks.

Feb 21 2024, 08:03 · Discussion Needed, Diffusion

Feb 20 2024

avivey added a comment to Q111: How to disable noreply mailbox? (Answer 143).

Ok, see here: https://we.phorge.it/book/phorge/article/configuring_outbound_email/#outbound-quot-from-quot-and-quot

Feb 20 2024, 07:30
sicenul added a comment to Q111: How to disable noreply mailbox? (Answer 143).
  • I have setup cluster.mailers using smtp type (phorge-mailer@gmail.com (example only))
  • those messages are receive in phorge-mailer@gmail.com from google mailer, informing that noreply@phorge.domain.com is not reachable so it bounce.
  • I assume, it happened because phorge send outbound email:
    • using noreply@phorge.domain.com as "To" address
    • "cc" address to recipients
    • "From" with format "Username (Fullname) <phorge-mailer@gmail.com>"
  • Is it possible to disable or change "To" address other than noreply@phorge.domain.com, because I didn't setup phorge.domain.com to receive email.
Feb 20 2024, 03:22

Feb 19 2024

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

btw, for the "search" use-case I'm describing, I'm imagining something like a Function on Users/project, so I can search, for example "Tasks that are assigned to users which I flagged with a green flag" or "Tasks that have a tag that I flagged with a blue flag".
And also, "Revisions that I flagged with any flag".

Feb 19 2024, 21:59
avivey added a comment to T15512: Make Flags more useful.

I looked into this again.

Feb 19 2024, 21:54
avivey added Q111: How to disable noreply mailbox? (Answer 143).
Feb 19 2024, 19:59
sicenul asked Q111: How to disable noreply mailbox?.
Feb 19 2024, 11:11

Feb 17 2024

avivey edited the content of How to Write Extensions.
Feb 17 2024, 13:33 · Extension Development
avivey created an object: How to Install Extensions.
Feb 17 2024, 13:32 · Extension Development
20after4 awarded R11:741e2946b93f: license a Like token.
Feb 17 2024, 12:52
avivey committed R11:741e2946b93f: license.
license
Feb 17 2024, 11:13
avivey committed R11:f9e20362b385: Fix nestes lists and work on Inputs too.
Fix nestes lists and work on Inputs too
Feb 17 2024, 11:13
avivey committed R11:b77da03b97ce: Create lightweight RTL extension.
Create lightweight RTL extension
Feb 17 2024, 11:13
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?

Feb 17 2024, 10:46 · UX, Accessibility
Iniquity awarded T15056: Improve Dark Mode a Love token.
Feb 17 2024, 10:36 · User-Cigaryno, User-valerio.bozzolan, Accessibility, User-Bukkit

Feb 16 2024

Wikivu resigned from rPaf06bfb1cc49: Make Changeset ID for render cache explicit.

Wiki volkankaos

Feb 16 2024, 16:07
Wikivu added 1 auditor(s) for rPaf06bfb1cc49: Make Changeset ID for render cache explicit: Wikivu.
Feb 16 2024, 16:06
bekay added a comment to T15048: Allow awarding Tokens to individual Comments.

Just asking for a friend: Is there some progress on this?

Feb 16 2024, 12:43 · Comments
bekay updated subscribers of Q108: Phorge and Mermaid Js (Answer 139).

Maybe this could be integrated in the Diagrams extension...? I mean: the entire infrastructure is there - you just have to decide, what lib you wanne use when creating a new diagram. @mturdus What do you think about this?

Feb 16 2024, 12:35
avivey created an object: How to Write Extensions.
Feb 16 2024, 11:05 · Extension Development
avivey created an object: Extensions.
Feb 16 2024, 11:05
valerio.bozzolan added a comment to Q110: Non working URL for query overheated explanation page.

It would be a pleasure to receive a patch from you :) The file should be this one:

Feb 16 2024, 11:00
valerio.bozzolan updated the question details for Q110: Non working URL for query overheated explanation page.
Feb 16 2024, 10:57
avivey added Q110: Non working URL for query overheated explanation page (Answer 142).
Feb 16 2024, 10:57
sicenul changed the visibility for F1375625: image.png.
Feb 16 2024, 10:40
sicenul changed the visibility for F1375613: image.png.
Feb 16 2024, 10:39
sicenul asked Q110: Non working URL for query overheated explanation page.
Feb 16 2024, 10:39
avivey edited the content of Community Resources.
Feb 16 2024, 10:25
avivey created an object: Community Resources.
Feb 16 2024, 10:06
avivey added a comment to D25535: T15011: Update support for XHPast on Windows.

Maybe this is wild, but: should we consider removing the compiled xhpast.exe file from the repository, and host is externally?
I think it's only used for working on PHP on Windows; The equivalent xhpast isn't included, presumably because compiling stuff on Windows is harder.
My argument for this is that the normal way to install arcanist is git clone, which downloads all the history (and 95% of users won't need this particular file).

Feb 16 2024, 08:08

Feb 15 2024

valerio.bozzolan added inline comments to D25535: T15011: Update support for XHPast on Windows.
Feb 15 2024, 23:48
valerio.bozzolan added inline comments to D25535: T15011: Update support for XHPast on Windows.
Feb 15 2024, 23:47
valerio.bozzolan accepted D25535: T15011: Update support for XHPast on Windows.

Passed a week. Nobody commented. I blindly trust you in this field :3 Approve.

Feb 15 2024, 23:46
valerio.bozzolan accepted D25537: Update PHP versions in install guide: Remove 5, Add 8..

sgtm

Feb 15 2024, 23:43
valerio.bozzolan edited the content of Next Up.
Feb 15 2024, 23:15
bekay attached a referenced file: F1360344: image.png.
Feb 15 2024, 19:01
bekay attached a referenced file: F1368592: image.png.
Feb 15 2024, 19:01
bekay added a comment to T15673: Phorge participation at FOSDEM 2024.

TRUST ME - YOU DON'T WANT TO LISTEN MY ENGLISH :D :D

Btw now the video is here :D :D AGAIN SORRY PHORGE FOR MY MISTAKES <3 <3

https://video.fosdem.org/2024/h2215/fosdem-2024-3610-unpack-phabricator-welcome-phorge-forking-the-opinionated-open-source-project-manager.av1.webm

Feb 15 2024, 18:59 · Blessed Communicators, Phorge
bekay closed D25536: Implements copy button in clone repo modal.
Feb 15 2024, 18:38
bekay committed rP89a5d3132c94: Implements copy button in clone repo modal.
Implements copy button in clone repo modal
Feb 15 2024, 18:38
bekay updated the diff for D25536: Implements copy button in clone repo modal.
Feb 15 2024, 18:35
valerio.bozzolan added a comment to D25536: Implements copy button in clone repo modal.

Premising that I've already accepted I noticed also this minor possible thing:

Feb 15 2024, 16:55
valerio.bozzolan accepted D25536: Implements copy button in clone repo modal.

Lovely thanks :)

Feb 15 2024, 16:49
bekay added a comment to D25536: Implements copy button in clone repo modal.

I really have struggled to find any device in our company that can't do await/async. I strongly bevlieve it is a non issue. Let us land this thing now and see if we get any feedback of an error.

Feb 15 2024, 16:29
bekay updated the diff for D25536: Implements copy button in clone repo modal.
  • Changes the order of the variables and function calls
Feb 15 2024, 16:29
bekay updated the diff for D25536: Implements copy button in clone repo modal.
  • Extends select behavior and only select content on first click
Feb 15 2024, 16:23
avivey added a comment to D25536: Implements copy button in clone repo modal.

The ".then()" part is pure hackness; The async/await is the sane way to do async programming.

Feb 15 2024, 10:39
bekay added a comment to D25536: Implements copy button in clone repo modal.

Again, non-blocking opinion:

Are we aware that the "async" keyword (that was never used in Phorge) may just cause a crash by syntax error (almost-immediately even without clicking?) by that kind of browsers, so, breaking the page, just because we want to provide a copy function and we want to avoid .then()?

I think we can introduce breaking changes, but it's strange to do that, now, with this minor feature.

If we are aware of this minor risk, indeed let's land.

Feb 15 2024, 10:08
valerio.bozzolan added inline comments to D25536: Implements copy button in clone repo modal.
Feb 15 2024, 10:07
valerio.bozzolan added a comment to D25536: Implements copy button in clone repo modal.

Again, non-blocking opinion:

Feb 15 2024, 09:43
bekay added a comment to D25536: Implements copy button in clone repo modal.

Additional non-blocking thing:

Maybe should be copy(text) with var data defined outside

Bonus point: copy() not defined during every call of JX.Stratcom.listen, but outside (?)

Feb 15 2024, 09:23
valerio.bozzolan added a comment to D25536: Implements copy button in clone repo modal.

Additional non-blocking thing:

Feb 15 2024, 09:13
valerio.bozzolan awarded D25536: Implements copy button in clone repo modal a Like token.
Feb 15 2024, 09:08
bekay added a comment to D25536: Implements copy button in clone repo modal.

Yes, we can avoid async.

So, this is just a tip, to avoid async and also to have everything a little more modular for no reason:

  1. remove async from copy()
  2. rename copy() to copyWithoutFeedback() and have the function that always return a Promise
    • so copyWithoutFeedback() has return navigator.clipboard.writeText(text); to always return a Promise
    • so copyWithoutFeedback() can use copyWithFeedback() but after that it return Promise.resolve(true) to always return a Promise
    • so copyWithoutFeedback() can fail with just return Promise.reject() to always return a Promise
  3. so we can create the function with feedback, copyWithFeedback(), that just calls the previous one, so, copyWithoutFeedback().then(function() { show_success_message() } ); with also a chained .fail( function() { show_error_message(); } ) or something similar
Feb 15 2024, 08:52
bekay updated the diff for D25536: Implements copy button in clone repo modal.
  • Updates map
Feb 15 2024, 08:36