Welcome in Phorge and long life to FreeBSD! You are now in the family of Trusted Contributors and so you can create Tasks.

Right. I mentioned that in at least one answer. I'm closing since you resolved :)

Maybe we can wontfix this, and try other more modern approaches like flatpak. Example:

It makes sense thanks. Minor note:

In what case, a class can be installed "for the viewer", but generally uninstalled?

double-accept

Sorry for the confusion. I mean that the /mail/ page is the Mail Dashboard that allows to see both Inbox and Outbox (visible from the left sidebar menu). For example this is the exact Outbox sub-page:

Indeed :)

Nice trick for you: to register a new identity try adding +something after your email username. Example: smith+test1@example.com. That is a secret feature of mailboxes.

Have you already seen this page? maybe useful?

(By the way the Filesystem::readFile() does not return a pointer to a (potentially read-only) resource, but it just returns its content - so the concern was nonsense)

In D25515#14983, @aklapper wrote:

In D25515#14842, @valerio.bozzolan wrote:

I wonder if it then, upon deletion of the profile image, it attempts to delete the builtin image

@valerio.bozzolan How is attempting to delete a built-in (!) profile image related to this patch? If I misunderstood, please elaborate. Thanks!

Nice. Maybe a start-extension Arcanist workflow is a bit overkill, but a boilerplate extension would be lovely

Thanks again. Since this is a WIP, let's mark as such.

In an ideal world 1030-1313 could become just:

I've read your awesome travel in T15659 but I've also read this inline comment:

It's nice and it works. So... thanks!

Uh, right. So maybe we can:

I cannot test it now, can you check who was setting that margin? Maybe we can try to avoid that !important

Good question. Maybe also related to L2.

Yes. I think we should wait for a kind Administrator to implement the first one.

Tested and I like it. Thanks :3

Uh thanks! You are now co-speaker

BTW I like the proposed screenshot. So I also agree to: feel free to share the solution 👍

Some help is needed there, testing on PHP 8.1+ and getting the right stack trace

I wonder if it then, upon deletion of the profile image, it attempts to delete the builtin image

Useful riepilogue for wild integers

I'm just unsure between implode() or phutil_implode_html()

Thanks again for this change. Please try "landing" with these two commands:

Do you mean "action"? If yes, I can reproduce

Indeed this fixes the problem. Premising there is still some mystery here that may deserve additional eyes.

Sentence case <3

Part of this Diff has no sense now that I learned that... some of these has sense. Let's close.

Double-accept yeah

(Small tip: never work in master. Create a git branch for each feature, in sync with origin/master, and with your commits on top of that, so your patch is minimal and arc diff works)

Yeah I honestly don't know if uppercase or lowercase.

I sincerely think this is lovely. Thanks. I do not understand how you figure it out all these keywords and in what order and associated to what, but hey, it works.

I set this to unbreak now just to give extra visibility on the homepage :^)

Ah, before landing, please consider an additional line in the comment (like the proposed one)

As far as I can understand, the generated file name here is 100% for commodity purposes: not used for anything on the filesystem, not used as unique key in something, not used to check whenever the file should be created again or not.

I'm testing this. It works and this makes me happy. Please wait Sunday before landing this, so others can give an eye.

Can you try changing this line in your local install?

Probably causing problems with Facebook OAuth

Run arc diff from my computer that has Xdebug enabled

Completely agree on lowering their ranking as default

I do not remember who was developing Docker stuff for this. I've just found again this

OH MY GOD FOR SOME REASONS THEY RE-OPENED OUR PROPOSAL \o/ IT'S HAPPENING!

Just discovered this from secure dot, 2017

This works as long as getFieldValue() never returns an object (that has a toString, that may be empty)