Thanks Cigaryno. I think it's good to start a discussion on this and I appreciate you started this. I also have seen github.com suggested somewhere.

Thanks. I'm curious if there's anything up that should not call this with null. Also I'm curious to test with a couple of date filters and custom date fields.

LGTM

I found this, which suggests that this shouldn't be used for headers. https://secure.phabricator.com/D21862

LGTM!

Thanks! I'm sorry that the execute method has undocumented return.

Thanks! I share a comment on the fly, maybe more comment soon after a test :)

I think you are right for unnecessary catetorization and sending notifications.

Running arc liberate before installing worked well for me. Thank you again!

OK super! <3

( I was running arc lint in phorge.git and got the same error.
Thank u for your recommendation of running arc liberate. Will add that to packaging script :)

Thaanks!

apply suggestions

(1. Are you an heroic package manager? <3 nice to know!)

I see the PHP approach is interestingly in line with stuff like line 11. wow lol. Thanks

none

One test is failing for PHP 8 (:
Lints are skipped because the same problem make linting failed for me.

Cigaryno moved this task from Backlog to PHP 8.1 on the PHP 8 support board.

Uhm, I realize this might not be a good idea because some might still be accessed by reflection.

Superseded by https://we.phorge.it/rPa4948ec800e9e385ecb519d86b40c5f13719fb53

I do not recommend, because https://phabricator.wikimedia.org/T293139 - Phorge displays whatever it gets, especially if the video got deleted in the meantime. Same behavior for Youtube.

Whopsie whopsie... you are now in the family of trusted contributors \o/

@valerio.bozzolan looks to not change behavior in mercurial at all :) I'm not in Trusted Contributors so I can't accept.

Do you think that this is wontfix? I think yes after https://we.phorge.it/T15281#10042

P.S. - finally a Mercurial user! Please try D25823 in Mercurial :D if it works for you, feel free to accept, or share any feedback

(Don't worry, I'm also super confused by arc now. lol - it still does not apply cleanly to me, so I've downloaded the patch...)

In D25865#23283, @valerio.bozzolan wrote:

Maybe we need to rebase on top of 7955fe6df701 (I don't have the base af404703cbe9 locally)

Rebase

Maybe we need to rebase on top of 7955fe6df701 (I don't have the base af404703cbe9 locally)

Thaaanks taavi! Known non-issue: somebody may wonder why in Maniphest the "Queries" are listed before other things. I may answer that in Phame the "Queries" are listed after other things. So this is consistent with Phame /phame/post/ page. lol

In D25863#23243, @valerio.bozzolan wrote:

If you want to easily try that missing piece I can add it with a little cute amend (keeping your authorship). We can always rollback the patch if you don't like the result or if it creates a nuclear weapon. Sounds OK?

Add searches to navigation menu

Yeah that makes sense, thanks!

If you want to easily try that missing piece I can add it with a little cute amend (keeping your authorship). We can always rollback the patch if you don't like the result or if it creates a nuclear weapon. Sounds OK?