Page MenuHomePhorge
Feed All Stories

Thu, Mar 20

Cigaryno added a comment to T15369: Clarify the "Do nothing" Herald Action.

On Outbound Email rules, does the Do Nothing action neither send an email nor a notification?

Thu, Mar 20, 21:24 · Herald
AndrewTavis updated AndrewTavis.
Thu, Mar 20, 19:40
valerio.bozzolan added a member for Trusted Contributors: AndrewTavis.
Thu, Mar 20, 19:13
Cigaryno added a project to T15424: Owners: Unhandled Exception: No repository PHID for path "0"!: Owners.
Thu, Mar 20, 14:39 · Owners, Bug Reports
Cigaryno added a project to T15083: Lipsum randomly fails when there are no repositories defined: Owners.
Thu, Mar 20, 14:39 · Owners
Cigaryno added a project to T15664: Bug: Unable to add an empty repository to an owners package: Owners.
Thu, Mar 20, 14:39 · Owners, Bug Reports, Diffusion
Cigaryno added a project to T16010: Owners not triggering audit on No Owner Involvement: Owners.
Thu, Mar 20, 14:39 · Good Starter Task, Owners, Bug Reports
Cigaryno created Owners.
Thu, Mar 20, 14:38
valerio.bozzolan attached a referenced file: F3271029: Screenshot Capture - 2025-03-20 - 08-37-59.png.
Thu, Mar 20, 10:40 · Phorge
valerio.bozzolan created T16017: Notifications: Mark All Read is disabled for some ghosts unread notifications.
Thu, Mar 20, 10:40 · Phorge
valerio.bozzolan added a comment to T16016: Phantom notification.

Uh interesting - good to know for the future. So at least the button Mark All Read here in F3271029 should be visible for your case - as very minimal mitigation sub-task

Thu, Mar 20, 10:33 · Maniphest
Tgr added a comment to T16016: Phantom notification.

The mark all read link in the notifications dropdown panel is always available. (It did clear the phantom notification.)

Thu, Mar 20, 10:30 · Maniphest
valerio.bozzolan added a comment to T16016: Phantom notification.

The current workaround is to wait for a real notification, to then finally be able to click the Mark All Read.

Thu, Mar 20, 09:20 · Maniphest
valerio.bozzolan added a comment to T16016: Phantom notification.

It happened to me but only in Wikimedia Phabricator, there I had a notification counter in the top-bar, but with no unread messages in the related list

Thu, Mar 20, 09:17 · Maniphest
avivey added a comment to T16016: Phantom notification.

I don't understand the problem described...

Thu, Mar 20, 09:15 · Maniphest
Tgr added a comment to T16016: Phantom notification.

Search brings up rP70d67a3908b8: Fix the most significant "phantom notification" badness

Thu, Mar 20, 07:47 · Maniphest
Tgr created T16016: Phantom notification.
Thu, Mar 20, 07:47 · Maniphest

Wed, Mar 19

valerio.bozzolan updated the task description for T15242: /config/ can run git log commands that silently fail ("fatal: detected dubious ownership in repository at ....").
Wed, Mar 19, 21:14 · Config
valerio.bozzolan awarded Q176: The hierarchy of documents is located after the document in the wiki a Like token.
Wed, Mar 19, 21:09 · UX
valerio.bozzolan accepted D25909: Diviner: Contributing Code: Update section on Prototype Changes.

Thaanks - please wait 7 days + 1 hour + 1 second + 42 milliseconds before landing to attract more Pòkèmòn reviewers

Wed, Mar 19, 13:57
Cigaryno added a comment to T15203: Remarkup: allow to mention a Slowvote (without embedding).

(M is short for Mockup probably)

Wed, Mar 19, 13:50 · User-valerio.bozzolan, Remarkup
valerio.bozzolan added a comment to T15203: Remarkup: allow to mention a Slowvote (without embedding).

(M is short for Mockup probably)

Wed, Mar 19, 09:42 · User-valerio.bozzolan, Remarkup
avivey added a comment to T16007: Discuss the policy for contributing to prototype applications.

I'm not familiar with MediaWiki's packages - the model I'm copying is VSCode.
My thought is that in the install manual we'll say "now run ./bin/extensions install phorge-recommended-extensions" (near the ./bin/storage) step, and phorge-recommended-extensions would be the equivalent of "extension pack" hosted on the default Extension Store, which is hosted here.
(VSCode also has "bundled extensions", which I think doesn't work for us because we use "clone the repo" as the primary distribution system).

Wed, Mar 19, 09:10 · Discussion Needed, User-Cigaryno
Cigaryno added a comment to T16007: Discuss the policy for contributing to prototype applications.

Ideally, any current Prototype can be either promoted to Core, extracted to its own extension, or removed completely. Each extension/author can have their own policy on contributing.

Already, any new app that would be considered "Prototype" today should just go in its own extension, and we decided to remove a couple.

It depends on who on the wild (including large private companies developing closed-source software) is using prototype applications on Phorge. This should let us know if it should be promoted to core, separated into an extension, or removed completely if no one uses it (like Releeph and Phragments). Or even better, hold a Slowvote for each prototype application's future and possibly have Phorge's customers to vote (maybe notify as much as possible by creating a blog post about the vote to notify those who use the Atom feed).

I'm not sure that "usage" is really the best way to choose between "promote to core" and "extension"; The way I imagine it, in addition to the Core, we'll have a set of "highly recommended extensions" maintained, and a single step to install all of them when setting up a new machine. In that world, any app that can be separated out to an extension will be.
The prototypes can usually be curved out easily, without effecting the rest of the code.

Wed, Mar 19, 08:52 · Discussion Needed, User-Cigaryno
avivey added a comment to T16007: Discuss the policy for contributing to prototype applications.

The "Prototype" concept was a way for Phacility to experiment with things without committing - but we have a different model today.

Really!? Phacility SaaS instances do not allow enabling prototypes and self-hosted Support (from the Support application on admin.phacility.com that was oddly marked as Prototype) likely wasn't even available for prototype applications.

Wed, Mar 19, 07:54 · Discussion Needed, User-Cigaryno
avivey added a comment to T15203: Remarkup: allow to mention a Slowvote (without embedding).

On this server, we changed the config to /^(Q|V|M)\d$/ because we don't have P1 tickets.
I'm also not sure what M is short for.

Wed, Mar 19, 07:40 · User-valerio.bozzolan, Remarkup
avivey added a comment to T15450: Configure another Ponder Question status "Closed, Graduated".

To me, "obsolete" manes "no longer appliable" - basically, "the information in this page doesn't apply any more".
Some (made up) examples:

  • "Can Phorge run on PHP 7.0?" (The answer is no, because of a specific bug in 7.0, which was EOL a long time ago)
  • "I have this problem in the Chatbot app" (We've deleted the chatbot app)
Wed, Mar 19, 07:34 · Feature Requests, Ponder, phorge.it install
Cigaryno added a comment to T15203: Remarkup: allow to mention a Slowvote (without embedding).

The V123 syntax is disabled by remarkup.ignored-object-names config by default; The default is /^(Q|V|M|P)\d$/ (basically anything starting with Q, V, M, or P), for "Q1" (biz-talk for April), "V1" (for versions), "M1" (for ?????) and "P1" (Jira for "important bug").

I'm honestly surprised about layout=inline working for {T123} - I thought is only works for images. But it can probably be made to work for Votes (or rather, all objects) like it does for tasks.

Wed, Mar 19, 07:19 · User-valerio.bozzolan, Remarkup
avivey added a comment to T15203: Remarkup: allow to mention a Slowvote (without embedding).

The V123 syntax is disabled by remarkup.ignored-object-names config by default; The default is /^(Q|V|M|P)\d$/ (basically anything starting with Q, V, M, or P), for "Q1" (biz-talk for April), "V1" (for versions), "M1" (for ?????) and "P1" (Jira for "important bug").

Wed, Mar 19, 07:07 · User-valerio.bozzolan, Remarkup

Tue, Mar 18

Cigaryno moved T15450: Configure another Ponder Question status "Closed, Graduated" from Backlog to New Features on the Feature Requests board.
Tue, Mar 18, 21:44 · Feature Requests, Ponder, phorge.it install
Cigaryno created T16015: Ponder: allow to quote comments.
Tue, Mar 18, 21:39 · Ponder, User-Cigaryno
Cigaryno moved T16014: Checkboxes are not processed on tables from Backlog to Feature Requests on the User-Cigaryno board.
Tue, Mar 18, 19:53 · Remarkup, User-Cigaryno
Cigaryno updated the task description for T16014: Checkboxes are not processed on tables.
Tue, Mar 18, 19:53 · Remarkup, User-Cigaryno
Cigaryno awarded T15203: Remarkup: allow to mention a Slowvote (without embedding) a Like token.
Tue, Mar 18, 19:50 · User-valerio.bozzolan, Remarkup
aklapper added a comment to Q176: The hierarchy of documents is located after the document in the wiki.

I now realize that this is not about hierarchy (documents below other documents) but about the Table of Contents within a single wiki page? You may want to edit the title.

Tue, Mar 18, 09:58 · UX

Mon, Mar 17

Cigaryno added a comment to Q176: The hierarchy of documents is located after the document in the wiki.

What you see is 100% normal regardless of your pixel density, display size and resolution.
With a similar approach to T15920, this can be achieved. However, I think there is one disadvantage: text may be crammed with a side hierarchy, resulting in lots of newlines for documents with long titles.
You can create a task now that you are member of Trusted Contributors.

Mon, Mar 17, 20:59 · UX
Cigaryno added a member for Trusted Contributors: vabocharov.
Mon, Mar 17, 20:59
Cigaryno added a comment to D25889: Add "Authored Tasks" view / item to user profile menu.

Hmm, maybe should not change ->setURI("/people/tasks/{$id}/") because there might be external third-party code relying on this?

Mon, Mar 17, 20:27
aklapper added Q176: The hierarchy of documents is located after the document in the wiki (Answer 216).
Mon, Mar 17, 20:10
aklapper added a comment to T16007: Discuss the policy for contributing to prototype applications.

It depends on who on the wild (including large private companies developing closed-source software) is using prototype applications on Phorge.

See T15501: Voluntary Usage Survey App basically.

Or even better, hold a Slowvote

Please no popularity contests (with even higher self-selection bias)...

Mon, Mar 17, 20:06 · Discussion Needed, User-Cigaryno
aklapper added a comment to D25889: Add "Authored Tasks" view / item to user profile menu.

Hmm, maybe should not change ->setURI("/people/tasks/{$id}/") because there might be external third-party code relying on this?

Mon, Mar 17, 19:58
vabocharov added a comment to Q176: The hierarchy of documents is located after the document in the wiki.

Changed the policy for file F3250825 to Public.

Mon, Mar 17, 19:53 · UX
vabocharov attached a referenced file: F3250825: image.png.
Mon, Mar 17, 19:51 · UX
Cigaryno added a comment to Q176: The hierarchy of documents is located after the document in the wiki.

@vabocharov please set the view policy of F3250825 to Public.

Mon, Mar 17, 19:38 · UX
vabocharov added a comment to Q176: The hierarchy of documents is located after the document in the wiki.

Good afternoon!
I get it, if you go to Phriction > Welcome to the Forge Wiki, then the hierarchy of documents will be displayed at the bottom (the screenshot below is attached). You won't see this in your example, as there are no attached documents.

Mon, Mar 17, 17:43 · UX
Cigaryno added a comment to T16007: Discuss the policy for contributing to prototype applications.

My thought on this is that long term, we'll remove the concept of "prototype" completely in favor of Extensions.

Prototypes that need a long way before being promoted to Core are those that should be separated into extensions.

Mon, Mar 17, 16:24 · Discussion Needed, User-Cigaryno
nib added a comment to D25872: Implements arc work for git workflow.
In D25872#23861, @nib wrote:

@aklapper Would it make sense to move the branch-naming lines into a function in class ArcanistGitWorkEngine, since it the branch-naming is git-specific? E.g.

public function buildBranchName($task_ref) {
  $task_name = $task_ref->getName();
  // regex, preg_replace, trim, etc
  return $branch_name;
}

Probably yes so it's easier with Subversion and Mercurial in the future (?)

Probably something with "for task" in the name like buildBranchNameForTask($task_ref)

Mon, Mar 17, 08:12
avivey updated the task description for T16007: Discuss the policy for contributing to prototype applications.
Mon, Mar 17, 07:39 · Discussion Needed, User-Cigaryno
avivey added a comment to T16007: Discuss the policy for contributing to prototype applications.

My thought on this is that long term, we'll remove the concept of "prototype" completely in favor of Extensions.
The "Prototype" concept was a way for Phacility to experiment with things without committing - but we have a different model today.

Mon, Mar 17, 07:39 · Discussion Needed, User-Cigaryno
nib updated the diff for D25872: Implements arc work for git workflow.
  • Handle arc work for new and existing git branches
Mon, Mar 17, 07:30
nib updated the diff for D25872: Implements arc work for git workflow.

Handle arc work for new and existing git branches

Mon, Mar 17, 06:38

Sun, Mar 16

Cigaryno added a comment to D25905: Remove No Prototype Changes section from Contributing Code document.

@Cigaryno: Oops, I am sorry! I had forgotten that this proposal also existed as it wasn't linked under "Revisions and Commits" in T16007. My fault. :-/

Sun, Mar 16, 21:02 · Documentation
aklapper added a comment to D25905: Remove No Prototype Changes section from Contributing Code document.

@Cigaryno: Oops, I am sorry! I had forgotten that this proposal also existed as it wasn't linked under "Revisions and Commits" in T16007. My fault. :-/

Sun, Mar 16, 20:54 · Documentation
aklapper removed a project from Q176: The hierarchy of documents is located after the document in the wiki: Affects-Wikimedia.
Sun, Mar 16, 20:51 · UX
aklapper added a comment to Q176: The hierarchy of documents is located after the document in the wiki.

Hej hej and welcome! I'm afraid I cannot really follow... In my understanding the hierarchy is expressed via the breadcrumbs navigation right below the top bar and not at the bottom, at least for a screen width of 513px and more?
For example if I go to https://we.phorge.it/w/changelog/next_up/ , see the Phriction > Welcome to the Phorge Wiki > Change Log > Next Up breadcrumbs.

Sun, Mar 16, 20:51 · UX
Cigaryno abandoned D25905: Remove No Prototype Changes section from Contributing Code document.

Obsoleted by D25909: Diviner: Contributing Code: Update section on Prototype Changes. I think @aklapper should have instead commandeered this rev but it's still okay to have a new revision instead.

Sun, Mar 16, 20:46 · Documentation
aklapper updated the diff for D25909: Diviner: Contributing Code: Update section on Prototype Changes.

where did my closing brakets go?

Sun, Mar 16, 20:42
aklapper requested review of D25909: Diviner: Contributing Code: Update section on Prototype Changes.
Sun, Mar 16, 20:40
aklapper added a revision to T16007: Discuss the policy for contributing to prototype applications: D25909: Diviner: Contributing Code: Update section on Prototype Changes.
Sun, Mar 16, 20:40 · Discussion Needed, User-Cigaryno
vabocharov updated the question details for Q176: The hierarchy of documents is located after the document in the wiki.
Sun, Mar 16, 19:41 · UX
vabocharov asked Q176: The hierarchy of documents is located after the document in the wiki.
Sun, Mar 16, 19:39 · UX
Cigaryno added a comment to T16007: Discuss the policy for contributing to prototype applications.

I do not think changes are necessarily needed, because it already says "With rare exceptions".

Bug fixes and security patches are indeed exceptions but not rare exceptions, assuming they fix problems with rough prototypes.

Sun, Mar 16, 16:29 · Discussion Needed, User-Cigaryno
20after4 awarded Q169: Reflecting diff progress on task status a Like token.
Sun, Mar 16, 13:21
mturdus accepted D25873: Fix overlapping elements in Phame for >1791px width.

sorry for the late reply.
This looks good to me.

Thanks for this patch! Kind reminder: if you touched CSS or JavaScript, please remember to also run this:

Sun, Mar 16, 08:58
valerio.bozzolan added a comment to T15450: Configure another Ponder Question status "Closed, Graduated".

I agree that an additional status is needed for closing when creating a task. Current statuses are not obvious

Sun, Mar 16, 00:01 · Feature Requests, Ponder, phorge.it install

Sat, Mar 15

valerio.bozzolan claimed T16001: Celerity unit test not triggered: mitigate thanks to Herald.

Uh! That's magic. Thanks avivey. Created here: H29 Phorge: add comment to remember "celerity map" - T15209

Sat, Mar 15, 23:58 · User-valerio.bozzolan, phorge.it install
valerio.bozzolan added Q166: Support for request header and body in "Make HTTP Request" step in Harbormaster (Answer 215).
Sat, Mar 15, 23:41
Cigaryno added a comment to T16001: Celerity unit test not triggered: mitigate thanks to Herald.

@valerio.bozzolan try to create the global rule now.

Sat, Mar 15, 16:19 · User-valerio.bozzolan, phorge.it install
avivey added Q166: Support for request header and body in "Make HTTP Request" step in Harbormaster (Answer 214).
Sat, Mar 15, 15:53
20after4 added a comment to Q166: Support for request header and body in "Make HTTP Request" step in Harbormaster.

I agree this seems pretty fundamental.

Sat, Mar 15, 15:00 · Almanac/Drydock/Harbormaster
20after4 awarded Q166: Support for request header and body in "Make HTTP Request" step in Harbormaster a Haypence token.
Sat, Mar 15, 14:59 · Almanac/Drydock/Harbormaster
20after4 added a comment to Q173: Why do Milestones in parent project's Workboard seemingly randomly hide tasks?.

I can’t think of a reason this would be happening other than a bug or database inconsistency.

Sat, Mar 15, 14:51
Cigaryno added a comment to U3 Installation Guide.

Fix the URL.

Sat, Mar 15, 10:25 · Phorge
Cigaryno changed the destination URL U3 Installation Guide from https://we.phorge.it/book/article/ to https://we.phorge.it/book/phorge/article/installation_guide/.
Sat, Mar 15, 10:25 · Phorge
Cigaryno created U3 Installation Guide.
Sat, Mar 15, 10:25 · Phorge
Cigaryno updated the task description for T15036: Phorge upstream mail should not use @secure.phorge.dev addresses.
Sat, Mar 15, 09:52 · phorge.it install
Cigaryno added a comment to T15020: Create a demo instance.

Sounds good. It should have no CAPTCHA configured (maybe we.phorge.it needs CAPTCHA to reduce unused account creations) and just like on secure.phabricator.com (see this), there should be a notice for users willing to demo the software on the demo instance and not the upstream instance (in Phabricator you can create a Phacility test instance and even to this day, this is still possible).

Sat, Mar 15, 09:25 · phorge.it install
Cigaryno awarded T15020: Create a demo instance a Like token.
Sat, Mar 15, 09:25 · phorge.it install
chris added a comment to T15566: Make is easier to install Phorge in the smallest case.

If there's interest, I revived hach-que's old Docker setup and am updating it to work with Phorge because azure devops is killing my soul at work. It's suitable for both local dev environments (T15011) and production installs, and I'll be supplementing the repo with IaC for a production deployment (T15928).

Sat, Mar 15, 02:17 · Phorge

Fri, Mar 14

Cigaryno created T16014: Checkboxes are not processed on tables.
Fri, Mar 14, 21:53 · Remarkup, User-Cigaryno
Cigaryno updated the task description for T15056: Improve Dark Mode.
Fri, Mar 14, 16:42 · User-Cigaryno, User-valerio.bozzolan, Accessibility, User-Bukkit
Sten closed T16012: DivinerArticleAtomizer.php fails with passing null to strlen error under PHP 8.1, a subtask of T15064: Make Phorge compatible with PHP 8.1/8.2/8.3/8.4, as Resolved.
Fri, Mar 14, 09:11 · PHP 8 support
Sten closed T16012: DivinerArticleAtomizer.php fails with passing null to strlen error under PHP 8.1 as Resolved by committing rP3e2d5e4bace1: Fix PHP8.1 strlen(null) error in DivinerArticleAtomizer.php.
Fri, Mar 14, 09:11 · PHP 8 support
Sten closed D25907: Fix PHP8.1 strlen(null) error in DivinerArticleAtomizer.php.
Fri, Mar 14, 09:11
Sten committed rP3e2d5e4bace1: Fix PHP8.1 strlen(null) error in DivinerArticleAtomizer.php.
Fix PHP8.1 strlen(null) error in DivinerArticleAtomizer.php
Fri, Mar 14, 09:11
aklapper closed T16013: Reports: x-axis labels overlap as Resolved by committing rPc8e3ee170f8a: Charts: Rotate x-axis labels for better readability.
Fri, Mar 14, 07:26
aklapper closed D25908: Charts: Rotate x-axis labels for better readability.
Fri, Mar 14, 07:26
aklapper committed rPc8e3ee170f8a: Charts: Rotate x-axis labels for better readability.
Charts: Rotate x-axis labels for better readability
Fri, Mar 14, 07:26
aklapper added a comment to D25908: Charts: Rotate x-axis labels for better readability.

Grazie milée!

Fri, Mar 14, 07:26

Thu, Mar 13

valerio.bozzolan edited the content of Next Up.
Thu, Mar 13, 22:37
valerio.bozzolan accepted D25907: Fix PHP8.1 strlen(null) error in DivinerArticleAtomizer.php.

sgtm

Thu, Mar 13, 22:35
valerio.bozzolan added a comment to D25908: Charts: Rotate x-axis labels for better readability.

Ah, it also works on my computer but really not relevant, just the patch description was in line with my business expectations

Thu, Mar 13, 22:29
valerio.bozzolan accepted D25908: Charts: Rotate x-axis labels for better readability.

This is EXACTLY the kind of serious-business patch description that should be mandatory in every Phorge patch to speedup code review. Thanks my friend. asd

Thu, Mar 13, 22:28
keithzg added a comment to D25723: PHPMailer: Merge class.phpmailer-lite.php into class.phpmailer.php.

I didn't notice anything amiss!

Thu, Mar 13, 21:08 · Mail
aklapper retitled D25908: Charts: Rotate x-axis labels for better readability from Reports: Rotate x-axis labels for better readability
Thu, Mar 13, 20:34
aklapper updated the summary of D25908: Charts: Rotate x-axis labels for better readability.
Thu, Mar 13, 20:21
aklapper updated the diff for D25908: Charts: Rotate x-axis labels for better readability.

Sure, let's try again, why not

Thu, Mar 13, 20:20
aklapper updated the diff for D25908: Charts: Rotate x-axis labels for better readability.

Make arc lint happier

Thu, Mar 13, 20:19
aklapper requested review of D25908: Charts: Rotate x-axis labels for better readability.
Thu, Mar 13, 20:17
aklapper added a revision to T16013: Reports: x-axis labels overlap: D25908: Charts: Rotate x-axis labels for better readability.
Thu, Mar 13, 20:17