Page MenuHomePhorge
Feed All Stories

Apr 22 2024

valerio.bozzolan added a revision to T15526: Diffusion: Description and tags are not shown: D25538: Separate repository home and browse view in tab menu.
Apr 22 2024, 05:09 · Good Starter Task, UX

Apr 21 2024

Jack_who_built_the_house added a comment to Q128: How deeply is Ponder integrated with the rest of Phorge? (Answer 159).

@20after4 Thanks for the response. Integration with search is something I overlooked, but you're right; tasks can naturally come from questions, and questions can naturally come from tasks. Their joint discoverability and interplay is a fertile soil for education and development. As volunteers, our role is kind of limited in many cases to being passive recipients of what is decided in WMF internal communications. Having public discussions of components could add more dynamism to the process.

Apr 21 2024, 21:20
20after4 added a comment to Q128: How deeply is Ponder integrated with the rest of Phorge? (Answer 159).

I think ponder is sort of a minimal viable product state currently. It lacks some features like putting questions on workboards and there just aren't many
(or any?) integrations with the rest of Phorge. That wouldn't take much effort to improve it, most likely.

Apr 21 2024, 19:19
Jack_who_built_the_house added a comment to Q128: How deeply is Ponder integrated with the rest of Phorge? (Answer 159).

But this is the exact reason I started to consider Ponder in the first place. All previous attempts implied fragmentation, while Ponder is integrated in the system people already use (Phabricator), both developers and volunteers, and in numbers. I've explained in more details how I see the current situation at https://meta.wikimedia.org/wiki/User_talk:SDeckelmann-WMF#How_volunteer_developer_support_could_be_improved_in_Wikimedia. Please share your view if you'd like. You can also join the discussion in the English Wikimedia Discord I started here.

Apr 21 2024, 13:28
aklapper added Q128: How deeply is Ponder integrated with the rest of Phorge? (Answer 159).
Apr 21 2024, 11:48
aklapper added a comment to T15526: Diffusion: Description and tags are not shown.

Which exact page(s) is this ticket about?

Apr 21 2024, 11:43 · Good Starter Task, UX
Iniquity awarded Q128: How deeply is Ponder integrated with the rest of Phorge? a Love token.
Apr 21 2024, 00:07 · Ponder, Phorge

Apr 20 2024

aklapper added a comment to D25080: Workboard Trigger Rule: allow to Add/Remove Subscribers.

Eeeh but in that way it's 84 and 87 chars long.

Apr 20 2024, 17:25 · Workboard
aklapper added a comment to D25601: Fix exception trying to rename user to their previous username.

Note that $xaction->getOldValue() in https://we.phorge.it/source/phorge/browse/master/src/applications/people/xaction/PhabricatorUserUsernameTransaction.php$62 seems to always return null. It does not provide the old/current username, while generateOldValue() actually does.

Apr 20 2024, 17:14
aklapper added a revision to T15795: Unhandled exception trying to rename user to their previous name: D25601: Fix exception trying to rename user to their previous username.
Apr 20 2024, 17:12
aklapper requested review of D25601: Fix exception trying to rename user to their previous username.
Apr 20 2024, 17:12
aklapper created T15795: Unhandled exception trying to rename user to their previous name.
Apr 20 2024, 17:07
Jack_who_built_the_house updated the question details for Q128: How deeply is Ponder integrated with the rest of Phorge?.
Apr 20 2024, 15:27 · Ponder, Phorge
Jack_who_built_the_house updated the question details for Q128: How deeply is Ponder integrated with the rest of Phorge?.
Apr 20 2024, 15:25 · Ponder, Phorge
Jack_who_built_the_house updated the question details for Q128: How deeply is Ponder integrated with the rest of Phorge?.
Apr 20 2024, 15:23 · Ponder, Phorge
Jack_who_built_the_house asked Q128: How deeply is Ponder integrated with the rest of Phorge?.
Apr 20 2024, 15:21 · Ponder, Phorge

Apr 19 2024

aklapper added a comment to D25600: Mark created Pholio mockups as new objects to fix empty Feed transaction entries.

Screenshot before (M17) and after (M18) applying D25600:

Screenshot from 2024-04-19 18-20-53.png (834×1 px, 162 KB)

Apr 19 2024, 16:31
aklapper requested review of D25600: Mark created Pholio mockups as new objects to fix empty Feed transaction entries.
Apr 19 2024, 16:30
aklapper added a revision to T15679: Pholio image upload triggers Passing null is deprecated at PhabricatorPolicyQuery.php:314: D25600: Mark created Pholio mockups as new objects to fix empty Feed transaction entries.
Apr 19 2024, 16:30 · PHP 8 support
aklapper requested review of D25599: Fix PHP 8.1 "strlen(null)" exception mass-editing tasks when custom Date field present.
Apr 19 2024, 15:56
aklapper added a revision to T15794: PHP 8.1 "strlen(null)" exceptions mass-editing tasks when custom Date field present: D25599: Fix PHP 8.1 "strlen(null)" exception mass-editing tasks when custom Date field present.
Apr 19 2024, 15:56 · PHP 8 support
aklapper created T15794: PHP 8.1 "strlen(null)" exceptions mass-editing tasks when custom Date field present.
Apr 19 2024, 15:30 · PHP 8 support
aklapper requested review of D25598: Fix exception for anonymous viewers on dashboard query panels with user-specific data.
Apr 19 2024, 11:52
aklapper added a revision to T15792: Dashboard query panel requiring a Current Viewer throws exception when not logged in: D25598: Fix exception for anonymous viewers on dashboard query panels with user-specific data.
Apr 19 2024, 11:52
aklapper requested review of D25597: Fix exception accessing a non-existing EditEngine.
Apr 19 2024, 10:58
aklapper added a revision to T15793: Exception trying to access non-existing EditEngine: D25597: Fix exception accessing a non-existing EditEngine.
Apr 19 2024, 10:58
aklapper created T15793: Exception trying to access non-existing EditEngine.
Apr 19 2024, 09:37
aklapper updated the task description for T15792: Dashboard query panel requiring a Current Viewer throws exception when not logged in.
Apr 19 2024, 08:37
aklapper created T15792: Dashboard query panel requiring a Current Viewer throws exception when not logged in.
Apr 19 2024, 08:33

Apr 18 2024

aklapper requested review of D25596: Fix PHP 8.1 "strlen(null)" exceptions editing existing Dashboard query panel with no query defined.
Apr 18 2024, 17:50
aklapper added a revision to T15791: PHP 8.1 "strlen(null)" exceptions editing existing Dashboard query panel with no query defined: D25596: Fix PHP 8.1 "strlen(null)" exceptions editing existing Dashboard query panel with no query defined.
Apr 18 2024, 17:50 · PHP 8 support
aklapper renamed T15791: PHP 8.1 "strlen(null)" exceptions editing existing Dashboard query panel with no query defined from PHP 8.1 "strlen(null)" exceptions editing existing Dashboard query panel to PHP 8.1 "strlen(null)" exceptions editing existing Dashboard query panel with no query defined.
Apr 18 2024, 17:47 · PHP 8 support
aklapper updated the task description for T15791: PHP 8.1 "strlen(null)" exceptions editing existing Dashboard query panel with no query defined.
Apr 18 2024, 17:44 · PHP 8 support
aklapper created T15791: PHP 8.1 "strlen(null)" exceptions editing existing Dashboard query panel with no query defined.
Apr 18 2024, 17:41 · PHP 8 support
aklapper created T15790: Exception creating a "Diffusion Raw Commits" Dashboard query panel.
Apr 18 2024, 17:40 · Diffusion, Dashboards
aklapper created T15789: Exception trying to export empty task list to CSV.
Apr 18 2024, 17:09
Mormegil added a comment to T15788: “Form data changed” confirmation is shown after even read-only keyboard interaction.

Yeah, that might help (like specifically in the case of the Clone dialog) even though it wouldn't help with dialogs containing writable inputs. (E.g. on the repository page, click Flag For Later instead and press Shift+Tab and Escape.) A better fix would lie at https://we.phorge.it/source/phorge/browse/master/webroot/rsrc/externals/javelin/lib/Workflow.js$413 where either the getSpecialKey would need improvement itself to return true for these non-modifying keys, or it would need to be amended with a custom check in that regard. However, I don't know where that function comes from.

Apr 18 2024, 15:40 · Bug Reports, Phorge
valerio.bozzolan added a comment to T15788: “Form data changed” confirmation is shown after even read-only keyboard interaction.

Probably something here should be expanded to check if the form contains at least a visible input without read-only

Apr 18 2024, 15:17 · Bug Reports, Phorge
Mormegil updated Mormegil.
Apr 18 2024, 15:15
Mormegil set Timezone to Europe/Prague on Mormegil.
Apr 18 2024, 15:13
Mormegil closed Q127: [bug] “Form data changed” confirmation is shown after even read-only keyboard interaction as obsolete.
Apr 18 2024, 15:11
Mormegil added a comment to Q127: [bug] “Form data changed” confirmation is shown after even read-only keyboard interaction.

Thanks, created as T15788

Apr 18 2024, 15:11
Mormegil created T15788: “Form data changed” confirmation is shown after even read-only keyboard interaction.
Apr 18 2024, 15:11 · Bug Reports, Phorge
valerio.bozzolan awarded Q127: [bug] “Form data changed” confirmation is shown after even read-only keyboard interaction a Yellow Medal token.
Apr 18 2024, 14:43
valerio.bozzolan added a comment to Q127: [bug] “Form data changed” confirmation is shown after even read-only keyboard interaction.

You are now in the family of Trusted Contributors 🎇

Apr 18 2024, 14:42
valerio.bozzolan added a member for Trusted Contributors: Mormegil.
Apr 18 2024, 14:40
Mormegil asked Q127: [bug] “Form data changed” confirmation is shown after even read-only keyboard interaction.
Apr 18 2024, 08:23

Apr 17 2024

valerio.bozzolan added a comment to D25475: Drag & Drop Task Cover Image: also attach.

Thanks again avivey. I liked the idea of using applyExternalEffects(). I have not found a way to do it efficiently, without introducing another static method.

Apr 17 2024, 16:10
aklapper closed D25592: Fix PHP 8.1 "strlen(null)" exception for "/bin/storage renamespace" without parameter.
Apr 17 2024, 11:12
aklapper committed rP5444e1c885c3: Fix PHP 8.1 "strlen(null)" exception for "/bin/storage renamespace" without….
Fix PHP 8.1 "strlen(null)" exception for "/bin/storage renamespace" without…
Apr 17 2024, 11:12
aklapper closed D25591: Fix PHP 8.1 "strlen(null)" exception for "/bin/almanac register" without parameter.
Apr 17 2024, 11:12
aklapper committed rP5fe2fc33fb1c: Fix PHP 8.1 "strlen(null)" exception for "/bin/almanac register" without….
Fix PHP 8.1 "strlen(null)" exception for "/bin/almanac register" without…
Apr 17 2024, 11:12
aklapper closed D25590: Fix PHP 8.1 "strlen(null)" exception for "/bin/aphlict notify" without parameter.
Apr 17 2024, 11:11
aklapper committed rPdb3e68137dcc: Fix PHP 8.1 "strlen(null)" exception for "/bin/aphlict notify" without parameter.
Fix PHP 8.1 "strlen(null)" exception for "/bin/aphlict notify" without parameter
Apr 17 2024, 11:11
aklapper closed D25588: Fix PHP 8.1 "strlen(null)" exception for "/bin/bulk export --query --format" without parameter.
Apr 17 2024, 11:10
aklapper committed rP85903f968fda: Fix PHP 8.1 "strlen(null)" exception for "/bin/bulk export --query --format"….
Fix PHP 8.1 "strlen(null)" exception for "/bin/bulk export --query --format"…
Apr 17 2024, 11:10
aklapper closed D25587: Fix PHP 8.1 "strlen(null)" exception for "/bin/auth cache-pkcs8" without parameter.
Apr 17 2024, 11:08
aklapper committed rP412c01ffa6fb: Fix PHP 8.1 "strlen(null)" exception for "/bin/auth cache-pkcs8" without….
Fix PHP 8.1 "strlen(null)" exception for "/bin/auth cache-pkcs8" without…
Apr 17 2024, 11:08
aklapper closed D25586: Fix PHP 8.1 "strlen(null)" exception for "/bin/herald test" without parameter.
Apr 17 2024, 11:08
aklapper committed rP0e53f91c9701: Fix PHP 8.1 "strlen(null)" exception for "/bin/herald test" without parameter.
Fix PHP 8.1 "strlen(null)" exception for "/bin/herald test" without parameter
Apr 17 2024, 11:08
aklapper closed D25584: Fix PHP 8.1 "strlen(null)" exception for "/bin/conduit call" without parameter.
Apr 17 2024, 11:07
aklapper committed rPd630f2281bcc: Fix PHP 8.1 "strlen(null)" exception for "/bin/conduit call" without parameter.
Fix PHP 8.1 "strlen(null)" exception for "/bin/conduit call" without parameter
Apr 17 2024, 11:07
valerio.bozzolan reassigned T15745: Cannot install extension "Diagrams" (Failed to load symbol "DiagramTransactionType") from valerio.bozzolan to bekay.

(Credit to bekay indeed)

Apr 17 2024, 08:05 · Diagrams (Extension)
valerio.bozzolan closed T15745: Cannot install extension "Diagrams" (Failed to load symbol "DiagramTransactionType") as Resolved.

Nice :) Thanks

Apr 17 2024, 08:04 · Diagrams (Extension)
valerio.bozzolan closed D25593: README: run upgrade after installation.
Apr 17 2024, 08:01
valerio.bozzolan committed R5:ce4b81b3705c: README: run upgrade after installation.
README: run upgrade after installation
Apr 17 2024, 08:01
valerio.bozzolan added a member for Diagrams (Extension): valerio.bozzolan.
Apr 17 2024, 08:01
aklapper closed D25595: Fix PHP 8.1 "strlen(null)" exception for "/bin/files encode" without --as parameter.
Apr 17 2024, 07:17
aklapper committed rP98cbf5fa19e2: Fix PHP 8.1 "strlen(null)" exception for "/bin/files encode" without --as….
Fix PHP 8.1 "strlen(null)" exception for "/bin/files encode" without --as…
Apr 17 2024, 07:17
aklapper closed D25594: Fix PHP 8.1 "strlen(null)" exception for "/bin/files migrate" without parameter.
Apr 17 2024, 07:16
aklapper committed rP0bdfcc47e7de: Fix PHP 8.1 "strlen(null)" exception for "/bin/files migrate" without parameter.
Fix PHP 8.1 "strlen(null)" exception for "/bin/files migrate" without parameter
Apr 17 2024, 07:16
valerio.bozzolan added a comment to D25569: Avoid separate per-line URIs in line number column.

Bonus point: maybe instead of "#123" we can use "#L123" since both GitLab and GitHub use that (and that's not a stupid specification). Just because we can.

Apr 17 2024, 07:09
valerio.bozzolan added a comment to D25578: Fix PHP 8.1 "strlen(null)" exceptions adding an OAuth provider.

If we find 10 minutes, let's test this on every provider, as last time it gave us a little surprise on one of them

Apr 17 2024, 06:59
valerio.bozzolan accepted D25595: Fix PHP 8.1 "strlen(null)" exception for "/bin/files encode" without --as parameter.

yesyes

Apr 17 2024, 06:57
valerio.bozzolan accepted D25594: Fix PHP 8.1 "strlen(null)" exception for "/bin/files migrate" without parameter.

yesyes

Apr 17 2024, 06:57

Apr 16 2024

aklapper requested review of D25595: Fix PHP 8.1 "strlen(null)" exception for "/bin/files encode" without --as parameter.
Apr 16 2024, 23:05
aklapper requested review of D25594: Fix PHP 8.1 "strlen(null)" exception for "/bin/files migrate" without parameter.
Apr 16 2024, 22:58
aklapper closed D25585: Fix PHP 8.1 "strlen(null)" exception for "/bin/herald rule" without parameter.
Apr 16 2024, 22:41
aklapper committed rPc59a883e9de7: Fix PHP 8.1 "strlen(null)" exception for "/bin/herald rule" without parameter.
Fix PHP 8.1 "strlen(null)" exception for "/bin/herald rule" without parameter
Apr 16 2024, 22:41
aklapper closed D25589: Fix PHP 8.1 "strlen(null)" exception for "/bin/search query" without parameter.
Apr 16 2024, 22:40
aklapper committed rPb29e5aa7ca22: Fix PHP 8.1 "strlen(null)" exception for "/bin/search query" without parameter.
Fix PHP 8.1 "strlen(null)" exception for "/bin/search query" without parameter
Apr 16 2024, 22:40
aklapper closed D25583: Fix PHP 8.1 "strlen(null)" exception for "/bin/auth refresh" without parameter.
Apr 16 2024, 22:38
aklapper committed rP102c43fb6a87: Fix PHP 8.1 "strlen(null)" exception for "/bin/auth refresh" without parameter.
Fix PHP 8.1 "strlen(null)" exception for "/bin/auth refresh" without parameter
Apr 16 2024, 22:38
aklapper closed D25582: Fix PHP 8.1 "strlen(null)" exception for "/bin/auth unlimit" without parameter.
Apr 16 2024, 22:37
aklapper committed rP2c7be156a8c3: Fix PHP 8.1 "strlen(null)" exception for "/bin/auth unlimit" without parameter.
Fix PHP 8.1 "strlen(null)" exception for "/bin/auth unlimit" without parameter
Apr 16 2024, 22:37
aklapper closed D25581: Fix PHP 8.1 "strlen(null)" exception for "/bin/nuance update" without parameter.
Apr 16 2024, 22:35
aklapper committed rP9101c0b63df4: Fix PHP 8.1 "strlen(null)" exception for "/bin/nuance update" without parameter.
Fix PHP 8.1 "strlen(null)" exception for "/bin/nuance update" without parameter
Apr 16 2024, 22:35
aklapper closed D25580: Fix PHP 8.1 "strlen(null)" exception for "/bin/nuance import" without parameter.
Apr 16 2024, 22:35
aklapper committed rP6a2f2a589a0f: Fix PHP 8.1 "strlen(null)" exception for "/bin/nuance import" without parameter.
Fix PHP 8.1 "strlen(null)" exception for "/bin/nuance import" without parameter
Apr 16 2024, 22:35
aklapper closed D25579: Fix PHP 8.1 "strlen(null)" exception for "/bin/files generate-key" without parameter.
Apr 16 2024, 22:29
aklapper committed rP05798a630120: Fix PHP 8.1 "strlen(null)" exception for "/bin/files generate-key" without….
Fix PHP 8.1 "strlen(null)" exception for "/bin/files generate-key" without…
Apr 16 2024, 22:29
bekay accepted D25593: README: run upgrade after installation.

You are right of course!

Apr 16 2024, 20:47
bekay added a comment to T15787: Diagrams: avoid double slash in the URL after Saving.

Yeah, there are some malformed URLs we should tackle for sure.

Apr 16 2024, 20:45 · Diagrams (Extension)
valerio.bozzolan triaged T15787: Diagrams: avoid double slash in the URL after Saving as Low priority.
Apr 16 2024, 20:21 · Diagrams (Extension)
valerio.bozzolan added a revision to T15745: Cannot install extension "Diagrams" (Failed to load symbol "DiagramTransactionType"): D25593: README: run upgrade after installation.
Apr 16 2024, 20:19 · Diagrams (Extension)
valerio.bozzolan retitled D25593: README: run upgrade after installation from README: run upgrade after installation Ref T15745
Apr 16 2024, 20:19
valerio.bozzolan added a reviewer for D25593: README: run upgrade after installation: bekay.
Apr 16 2024, 20:19
valerio.bozzolan added a comment to T15745: Cannot install extension "Diagrams" (Failed to load symbol "DiagramTransactionType").

Thanks! It worked now! As mentioned in the chat, I'm quite sure that point n.3 and n.4 should be swapped in the README

Apr 16 2024, 20:19 · Diagrams (Extension)
valerio.bozzolan requested review of D25593: README: run upgrade after installation.
Apr 16 2024, 20:19
bekay added a comment to Q108: Phorge and Mermaid Js (Answer 139).

I have edited the answer. Because with draw.io you can integrate Mermaid out of the box. With R5 you can start using the syntax like instantly.

Apr 16 2024, 18:30