Is this going to land in master any time soon? Our company would love to use the draft feature.

Also, in case of "arc land --merge", the merge commit is falsely authored by the revision author.

arc land still prints a warning in this scenario. Is there a way to disable that through .arcconfig? arc land itself doesn't seem to have an option to skip the warning.

We require the "a revision's reviewers can always..." behavior. Please implement it instead of changing the text.

@epriestley can you have a look at this, please?

@epriestley can you have a look at this, please?

Just for the record, this change works for me. Many thanks for quickly resolving my problem!

Alright, then this is it. I think a visual indication would be helpful. But apart from that, this can be closed.

So I asked a coworker to register with *his* credentials, and it turns out he can't see the email field. I strongly suspect it has to do with the fact that I already registered with the same email address when setting up the admin account initially.

Here is my ldap auth config

I tried again on a clean phabricator install (git master as of 10 minutes ago) on Ubuntu 16.04 and php7.1 from https://launchpad.net/~ondrej/+archive/ubuntu/php and still observe the same issue. So it looks like it's specific to our ldap server? Does the output I gave you above ring any bell?

Thank you for your extremely intensive efforts to reproduce my issue.

I tested this. Perhaps I'm missing something, but with this patch, when a new user registers using ldap, the email address field is NOT populated. I thought this was the purpose of this change?

I am also affected. Is there any known workaround?

How can I enable base64 encoding within phabricator?

The client was thunderbird and the webclient (based Open-Xchange). I received it on my personal mail account, outside the company.

Here's the mail that I received for your comment. You can see that the M stream is broken with "!" in two places. This seems to match the behavior I observed initially, although there was also an extraneous blank.

The webmail client allows me to view the raw mail. It's malformed as well.

bin/mail show-outbound --id <id> --dump-html shows the mail fine.

@epriestley this change works for us in de_DE locales. It would be nice if this was merged.

@epriestley I also get this error. What about your fix?

I encountered this on a svn repository, but what do you do for git repositoires where "label: one line summary" is considered best practice for commit messages?