Page MenuHomePhorge
Feed All Stories

Aug 19 2021

dcog added a comment to D25015: Show confirmation dialog when closing a modal if form contents have been changed.
In D25015#616, @speck wrote:

In this case I think the error text is agnostic of the instance of the dialog. From looking at AphrontDialogView I didn't see any obvious way to include additional fields/text that could be pulled out here on the front end. Looking elsewhere in this file (line ~297) it looks like some other generic text is used

if (!this._paused) {
  JX.$E('Resuming a workflow which is not paused!');
}

However that's the only case where JX.$E is used outside of an is-development check. I'm not sure where these Javascript translations would go or when the translation file is generated.

Aug 19 2021, 04:33 · Maniphest (archived)
codemouse92 added a comment to T15035: Discuss communications channels (IRC, etc).

To be fair, I'm almost certainly not going to install yet another client just for this community chat. (I know I'm not alone in this, projecting forward into the project's future.) Matrix + IRC with bridges to the other protocols seems the way to go. (I'm usually on IRC...albeit, not much this month because my plate's full.)

Aug 19 2021, 02:09 · phorge.it install
speck accepted D25019: Update Install and Update Scripts to the new Paths and Repository links.

This looks good to me.

Aug 19 2021, 01:41
speck added a comment to T15010: Vision Statement.

I added @dcog @codemouse92 and @mydeveloperday to the trusted contributors group. Glad to have everyone involved!

Aug 19 2021, 01:38 · Phorge, Governance
speck added members for Trusted Contributors: dcog, codemouse92, mydeveloperday.
Aug 19 2021, 01:37
speck added a comment to D25015: Show confirmation dialog when closing a modal if form contents have been changed.

In this case I think the error text is agnostic of the instance of the dialog. From looking at AphrontDialogView I didn't see any obvious way to include additional fields/text that could be pulled out here on the front end. Looking elsewhere in this file (line ~297) it looks like some other generic text is used

if (!this._paused) {
  JX.$E('Resuming a workflow which is not paused!');
}
Aug 19 2021, 01:34 · Maniphest (archived)
speck added a comment to D25018: Add conduit endpoints for querying legalpad.

I'm not familiar with the search engine or legalpad frameworks but things look like they line up

Aug 19 2021, 01:22

Aug 18 2021

roguelazer closed D25017: Fix PhutilLibraryMapBuilder to call the right function in log().
Aug 18 2021, 21:42
roguelazer committed rARC5407e5e5c67e: Fix PhutilLibraryMapBuilder to call the right function in log().
Fix PhutilLibraryMapBuilder to call the right function in log()
Aug 18 2021, 21:42
chris added a comment to D25017: Fix PhutilLibraryMapBuilder to call the right function in log().

@roguelazer - added you to Trusted Contributors so you should be able to land now

Aug 18 2021, 21:41
chris added a member for Trusted Contributors: roguelazer.
Aug 18 2021, 21:40
roguelazer added a comment to D25017: Fix PhutilLibraryMapBuilder to call the right function in log().

FWIW, I can't land this myself since I don't have push access to any repos; I would appreciate it if someone would either land it (or give me such permissions, if that's the preference).

Aug 18 2021, 20:16
20after4 added a comment to D25018: Add conduit endpoints for querying legalpad.

Wow, this is something I developed for wikimedia but not nearly as extensive! I'll try to give some code review when I have a few minutes to look this over.

Aug 18 2021, 18:09
20after4 added a comment to D25015: Show confirmation dialog when closing a modal if form contents have been changed.

+1 this has bitten me before.

Aug 18 2021, 18:08 · Maniphest (archived)
mydeveloperday added a comment to T15010: Vision Statement.

Not sure if this is where to mention, I would definitely like to help develop/maintain/review Phorge (I have signed the doc), I currently maintain a 600+ user Phabricator instance housing 50+ git repos,

Aug 18 2021, 17:08 · Phorge, Governance

Aug 17 2021

Chucky created Maro.
Aug 17 2021, 17:08
Chucky added a watcher for Daemons: Chucky.
Aug 17 2021, 16:40
Chucky awarded rP2258ba8535d5: Add package.json for aphlict a Y So Serious token.
Aug 17 2021, 13:54

Aug 15 2021

Leon95 updated the diff for D25019: Update Install and Update Scripts to the new Paths and Repository links.
  • Be quiet Linter!
Aug 15 2021, 13:50
Leon95 updated the diff for D25019: Update Install and Update Scripts to the new Paths and Repository links.
  • Move Links and Names to Variables
Aug 15 2021, 13:39

Aug 12 2021

Chucky added a watcher for Upstream General/Unknown: Chucky.
Aug 12 2021, 18:35
speck added a comment to D25019: Update Install and Update Scripts to the new Paths and Repository links.

The ubuntu/rhel scripts have been removed in the upstream (https://secure.phabricator.com/D21678). We can probably avoid making changes to those files as they should go away once we merge in changes.

Aug 12 2021, 01:29

Aug 11 2021

codemouse92 added a comment to T15010: Vision Statement.
In T15010#1062, @avivey wrote:

Any user should be able to sign the doc at L1 Phorge Vision Statement - did you try it and got an error?

Aug 11 2021, 20:43 · Phorge, Governance
avivey updated the task description for T15010: Vision Statement.
Aug 11 2021, 20:04 · Phorge, Governance
avivey added a comment to T15010: Vision Statement.

I agree with the doc, and would like to sign too.

Aug 11 2021, 20:03 · Phorge, Governance
Leon95 updated the test plan for D25019: Update Install and Update Scripts to the new Paths and Repository links.
Aug 11 2021, 19:50
Leon95 retitled D25019: Update Install and Update Scripts to the new Paths and Repository links from Update install and update Scripts to the new paths and repository links
Aug 11 2021, 19:45
Leon95 requested review of D25019: Update Install and Update Scripts to the new Paths and Repository links.
Aug 11 2021, 19:40
codemouse92 updated codemouse92.
Aug 11 2021, 16:57
codemouse92 added a comment to T15010: Vision Statement.

Glad to see Phabricator continuing as Phorge. Not going to lie, I had a panic attack this morning when I found out that Phacility was winding down.

Aug 11 2021, 16:55 · Phorge, Governance

Aug 8 2021

roguelazer updated roguelazer.
Aug 8 2021, 20:37
iamricard updated iamricard.
Aug 8 2021, 11:18

Aug 7 2021

eax accepted D25017: Fix PhutilLibraryMapBuilder to call the right function in log().
Aug 7 2021, 20:07

Aug 4 2021

roguelazer requested review of D25018: Add conduit endpoints for querying legalpad.
Aug 4 2021, 23:28
speck added a comment to D25017: Fix PhutilLibraryMapBuilder to call the right function in log().

The arcanist source in general seems to be split between fwrite(STDERR, $message."\n") and fprintf(STDERR, "%s\n", $message); I decided to go with the latter.

I think the latter format would be preferred as the formatting is more common elsewhere in the code, as opposed to concatenation.

Aug 4 2021, 00:37

Aug 3 2021

roguelazer requested review of D25017: Fix PhutilLibraryMapBuilder to call the right function in log().
Aug 3 2021, 18:14

Jul 30 2021

MacFan4000 added a comment to T15037: Support OAuth login via GitHub/Google/etc?.

And for google you don't need to provide an email, you can also just get a gmail address.

Jul 30 2021, 16:46 · Governance
MacFan4000 added a comment to T15037: Support OAuth login via GitHub/Google/etc?.

@speck we can use the existing GitHub organization for oauth. (an owner can setup the app in settings)

Jul 30 2021, 16:40 · Governance

Jul 29 2021

speck added a comment to T15037: Support OAuth login via GitHub/Google/etc?.

I think this would be reasonable but I think it requires setting up an account on GitHub or Google on behalf of Phorge (rather than using a member's personal account). If someone wants to set up a GitHub/Google account for phorge we can set it up here. It'll require an email address though and I'm not sure how to handle an organization email like that.

Jul 29 2021, 22:00 · Governance
20after4 created T15037: Support OAuth login via GitHub/Google/etc?.
Jul 29 2021, 21:53 · Governance
20after4 added a comment to T15014: Develop a Phorge Release Process.

I can help out with upstream merges. I've been doing it on a regular basis for Wikimedia for a long time now. It's rarely been a problem but I'm been careful to make sure that Wikimedia's fork doesn't drift too far away from upstream.

Jul 29 2021, 21:33 · Phorge

Jul 28 2021

MacFan4000 added a comment to T15035: Discuss communications channels (IRC, etc).

Just as a note, All of the mentioned chat platforms can be bridged together.

Jul 28 2021, 21:19 · phorge.it install
speck added a comment to T15035: Discuss communications channels (IRC, etc).

I'm not a big fan of IRC because in order to look through history or pickup where a user last joined it requires individual users to maintain their own tooling/practices. The temp-community chat was the first Zulip chat that I've used, and installed the mobile app for. I thought Zulip worked pretty well and wouldn't mind continuing with that. I've used Matrix (Element?) briefly but didn't feel too strongly about it. It's probably worth mentioning that our current Zulip instance has ~135 members and we've still had ~1-2 new people join per week lately. Though there's not much actual discussion going on there.

Jul 28 2021, 20:54 · phorge.it install
speck added a comment to T15014: Develop a Phorge Release Process.

Handling of merging upstream changes will probably lead to some challenging merges done by people who did not author the original changes in Phorge. I don't know what the best way is to work around this. Maybe if we regularly (daily?) pull in changes from upstream, merge into our own master branch which everyone lands onto that should mostly catch merge issues?

Jul 28 2021, 20:46 · Phorge
MacFan4000 added a comment to T15035: Discuss communications channels (IRC, etc).

We don't seem to have any opposition to having a community/support IRC channel, so perhaps I'll start working with Libera staff on registration.

Jul 28 2021, 17:58 · phorge.it install
avivey added a subtask for T15000: Ad hoc setup tasks: T15036: Phorge upstream mail should not use @upstream.phorge.dev addresses.
Jul 28 2021, 16:49 · Phorge.it Systems
avivey added a parent task for T15036: Phorge upstream mail should not use @upstream.phorge.dev addresses: T15000: Ad hoc setup tasks.
Jul 28 2021, 16:49 · phorge.it install
avivey awarded T15036: Phorge upstream mail should not use @upstream.phorge.dev addresses a Like token.
Jul 28 2021, 16:49 · phorge.it install
avivey added a comment to T15035: Discuss communications channels (IRC, etc).

I just want everything "important" to be happening here, where the search and cross-references works and async communication is easy.

Jul 28 2021, 16:48 · phorge.it install
taavi created T15036: Phorge upstream mail should not use @upstream.phorge.dev addresses.
Jul 28 2021, 15:38 · phorge.it install
taavi added a comment to T15035: Discuss communications channels (IRC, etc).

I prefer IRC but don't have any issues with having Matrix bridged there, but I would prefer to not have separate unbridged channels to reduce fragmentation of the already-small community.

Jul 28 2021, 15:07 · phorge.it install
MacFan4000 added a comment to T15035: Discuss communications channels (IRC, etc).

Are there any thoughts about a matrix channel? It would also be possible to bridge it to IRC.

Jul 28 2021, 15:05 · phorge.it install
revi added a comment to T15035: Discuss communications channels (IRC, etc).

Are there any thoughts about a matrix channel? It would also be possible to bridge it to IRC.

Jul 28 2021, 15:05 · phorge.it install
TitanNano added a comment to T15035: Discuss communications channels (IRC, etc).

Are there any thoughts about a matrix channel? It would also be possible to bridge it to IRC.

Jul 28 2021, 14:51 · phorge.it install
avivey added a comment to D25015: Show confirmation dialog when closing a modal if form contents have been changed.

The translated text can be a property of the dialogue itself, if we're worried about having extra objects laying around - the confirmation feels like it's a feature of the dialogue anyway.

Jul 28 2021, 07:20 · Maniphest (archived)
avivey added a comment to D25015: Show confirmation dialog when closing a modal if form contents have been changed.

There's no way to invoke pht from JS (because it's really complicated in the general case). The general practice is to provide the translated value from PHP when building the relevant objects. There should be an example in the UIExamples app - there's a page with buttons built in JS.

Jul 28 2021, 07:11 · Maniphest (archived)
avivey added a comment to T15014: Develop a Phorge Release Process.

There already are a bunch of worthwhile changes in Phabricator we should pick up; @Matthew - want to load them to check out the procedure? I can do that otherwise.

Jul 28 2021, 07:08 · Phorge
dcog added a comment to D25015: Show confirmation dialog when closing a modal if form contents have been changed.

I 100% agree with this. Is there no way to invoke internationalization via JS only? If not, that's perfectly great, but wondering about an answer offhand.

Jul 28 2021, 06:40 · Maniphest (archived)

Jul 27 2021

Matthew edited the content of Change Log.
Jul 27 2021, 15:45
Matthew renamed 2021 Week 36 (Early September) from 2021 Week 27 (Mid July) to 2021 Week 30 (Very Late July).
Jul 27 2021, 15:44
Matthew added a comment to T15001: Set up initial policies.

Just a note that Trusted Contributors can’t self grow beyond admins adding people currently as to add members you need to be able to edit the project. Currently only admins can edit the project.

IMO the edit policy should be set to Project Members. This way it’ll be able to self grow. :)

Jul 27 2021, 15:42 · Upstream General/Unknown
Matthew changed the edit policy for Trusted Contributors.
Jul 27 2021, 15:42
MacFan4000 added a comment to T15001: Set up initial policies.

Just a note that Trusted Contributors can’t self grow beyond admins adding people currently as to add members you need to be able to edit the project. Currently only admins can edit the project.

Jul 27 2021, 01:10 · Upstream General/Unknown

Jul 26 2021

Ekubischta added a comment to D25015: Show confirmation dialog when closing a modal if form contents have been changed.
Jul 26 2021, 22:31 · Maniphest (archived)
MacFan4000 added a comment to T15035: Discuss communications channels (IRC, etc).
In T15035#1008, @eax wrote:

FWIW I don't object to support or community channels off-instance but I do think we need:

  • all decisions should be made on-instance
  • we should have a culture of ensuring that any "interesting" discussions get recorded on-instance.
Jul 26 2021, 19:00 · phorge.it install
eax added a comment to T15035: Discuss communications channels (IRC, etc).

FWIW I don't object to support or community channels off-instance but I do think we need:

  • all decisions should be made on-instance
  • we should have a culture of ensuring that any "interesting" discussions get recorded on-instance.
Jul 26 2021, 18:26 · phorge.it install
MacFan4000 added a comment to T15035: Discuss communications channels (IRC, etc).

I would certainly be happy to work with Libera staff to get the community registered.

Jul 26 2021, 18:25 · phorge.it install
MacFan4000 renamed T15035: Discuss communications channels (IRC, etc) from Discuss wcommunications channels (irc, etc) to Discuss communications channels (IRC, etc).
Jul 26 2021, 18:23 · phorge.it install
MacFan4000 created T15035: Discuss communications channels (IRC, etc).
Jul 26 2021, 18:23 · phorge.it install
MacFan4000 updated MacFan4000.
Jul 26 2021, 17:05
dcog added a comment to T15010: Vision Statement.

I would like to be a part of the core team, and I've signed the document.

Jul 26 2021, 04:29 · Phorge, Governance

Jul 25 2021

dcog added a comment to D25015: Show confirmation dialog when closing a modal if form contents have been changed.

On a related note, tonight I discovered this UIExamples application (changing links from my Vagrant install):

Jul 25 2021, 04:07 · Maniphest (archived)
dcog added a comment to D25015: Show confirmation dialog when closing a modal if form contents have been changed.
In D25015#532, @speck wrote:

It’s generally more clear to have the buttons read “Discard changes” vs. “Ok” or “Yes”

I agree

Would it be easy to change the prompt to be more descriptive with actions?

For me, I think it might be on the hard side... and maybe add some bloat? Unless there is a styled generic confirm box created that maybe uses a callback instead of blocks execution...

Jul 25 2021, 03:30 · Maniphest (archived)
dcog updated the summary of D25015: Show confirmation dialog when closing a modal if form contents have been changed.
Jul 25 2021, 03:02 · Maniphest (archived)
Ekubischta awarded D25015: Show confirmation dialog when closing a modal if form contents have been changed a Like token.
Jul 25 2021, 02:55 · Maniphest (archived)

Jul 24 2021

speck added a comment to D25015: Show confirmation dialog when closing a modal if form contents have been changed.

Would it be easy to change the prompt to be more descriptive with actions? It’s generally more clear to have the buttons read “Discard changes” vs. “Ok” or “Yes”

Jul 24 2021, 21:04 · Maniphest (archived)
dcog requested review of D25015: Show confirmation dialog when closing a modal if form contents have been changed.
Jul 24 2021, 06:55 · Maniphest (archived)
dcog updated the task description for T15034: Show confirmation dialog when closing a modal if form contents have been changed.
Jul 24 2021, 06:47 · Maniphest, UX
dcog created T15034: Show confirmation dialog when closing a modal if form contents have been changed.
Jul 24 2021, 06:47 · Maniphest, UX

Jul 23 2021

MacFan4000 added a comment to D25014: (PhabricatorENV) update doclinks to link to we.phorge.it instead of secure.phabricator.com.

Thanks for accepting this. I can’t land it though, somebody with the proper permissions will need to do it.

Jul 23 2021, 23:46
deadalnix added a comment to D25014: (PhabricatorENV) update doclinks to link to we.phorge.it instead of secure.phabricator.com.

It is a step forward, but considering how the URL could be factorized would be a plus, IMO.

Jul 23 2021, 23:37
deadalnix accepted D25014: (PhabricatorENV) update doclinks to link to we.phorge.it instead of secure.phabricator.com.
Jul 23 2021, 23:36
MacFan4000 requested review of D25014: (PhabricatorENV) update doclinks to link to we.phorge.it instead of secure.phabricator.com.
Jul 23 2021, 23:04

Jul 20 2021

sario528 updated sario528.
Jul 20 2021, 18:56
MacFan4000 added a comment to T15030: Support a Phorge Extensions ecosystem.

regarding a monorepo, I'm not sure if there is an advantage to that, I'm fine with individual repos. I currently maintain most of Wikimedia's extensions in a single monorepo but I'd consider splitting them out into individual repos if any one them were candidates for upstreaming.

Jul 20 2021, 14:52 · Extension Development, Phactory: Community Projects, Phorge
MacFan4000 added a comment to T15001: Set up initial policies.

Just a note that Trusted Contributors can’t self grow beyond admins adding people currently as to add members you need to be able to edit the project. Currently only admins can edit the project.

Jul 20 2021, 02:18 · Upstream General/Unknown

Jul 19 2021

MacFan4000 added a member for Auth (archived): MacFan4000.
Jul 19 2021, 16:16
20after4 added a comment to T15030: Support a Phorge Extensions ecosystem.

regarding a monorepo, I'm not sure if there is an advantage to that, I'm fine with individual repos. I currently maintain most of Wikimedia's extensions in a single monorepo but I'd consider splitting them out into individual repos if any one them were candidates for upstreaming.

Jul 19 2021, 03:57 · Extension Development, Phactory: Community Projects, Phorge
20after4 added a comment to T15030: Support a Phorge Extensions ecosystem.

I'd like to host https://github.com/wikimedia/phabricator-antivandalism here, perhaps under a new name.

Jul 19 2021, 03:55 · Extension Development, Phactory: Community Projects, Phorge

Jul 18 2021

zeigren added a comment to T15025: Simple Production Docker Stack.

The Caddyfile boils down to

Jul 18 2021, 19:00
dcog added a comment to T15030: Support a Phorge Extensions ecosystem.
In T15030#972, @avivey wrote:

A space-zombie game that can report high-scores to IRC, Slack and Phorge.

Jul 18 2021, 07:06 · Extension Development, Phactory: Community Projects, Phorge

Jul 17 2021

avivey added a comment to T15030: Support a Phorge Extensions ecosystem.

Here's the terms I've been thinking of for accepting a project to Phactory:

  1. Directly related to Phorge in a significant way:

Good: Module for integration with some other system; New Phorge app that does something reasonable; scripts for administration work.
Bad: A space-zombie game that can report high-scores to IRC, Slack and Phorge.

Jul 17 2021, 07:23 · Extension Development, Phactory: Community Projects, Phorge
avivey updated the task description for T15022: Legal stuff.
Jul 17 2021, 06:50 · Phorge
gadgetsteve added a comment to T15033: Add option for GDPR Warning Suggestion.

Checking the Wikipedia entry for GDPR at https://en.wikipedia.org/wiki/General_Data_Protection_Regulation it mentions that this regulation or other similar ones have been enacted in:

Jul 17 2021, 06:08
gadgetsteve added a comment to T15033: Add option for GDPR Warning Suggestion.

Of course since Phorge itself is a public project with, potentially, UK/EU contributors the signup/login page should really display such a warning.

Jul 17 2021, 05:53
gadgetsteve created T15033: Add option for GDPR Warning Suggestion.
Jul 17 2021, 05:51

Jul 16 2021

MacFan4000 added a member for Upstream General/Unknown: MacFan4000.
Jul 16 2021, 17:50
speck added a comment to T15008: Build Welcome Site.

Yea I think it will be easiest to host this on the same server that's running we.phorge.it. If we're able to build it as an extension application for phabricator it can be setup in the same installation and then we can route the web server to host it properly.

Jul 16 2021, 13:33 · phorge.it install (DEPRECATED), Upstream General/Unknown
luca.itro added a comment to T15008: Build Welcome Site.

I think that the best choice is to keep everything in one place, so create the presentation site on the same infrastructure of we.phorge.it

Jul 16 2021, 13:24 · phorge.it install (DEPRECATED), Upstream General/Unknown
luca.itro added a comment to T15008: Build Welcome Site.

sorry for late reply. I can start working on it now. Where will be it hosted? Do you have some preference? We can use also my company infrastructure and change DNS record for a 3rd level to point to the IP.

Jul 16 2021, 13:13 · phorge.it install (DEPRECATED), Upstream General/Unknown