Page MenuHomePhabricator
Feed All Stories

Jul 6 2016

chad merged T11283: Make Task Graph Collapsable into T4788: Allow "Edit Dependencies" both ways (blocking and depending).
Jul 6 2016, 6:28 PM · Prioritized, FreeBSD, User Delight, Maniphest, Wikimedia
CodeMouse92 created T11283: Make Task Graph Collapsable.
Jul 6 2016, 6:06 PM · Maniphest, Bug Report
chad merged task T11282: Clearly indicate in email footer how to unsubscribe from similar emails into T9412: Annotate all the reasons that a user is included on the recipient list for mail.
Jul 6 2016, 5:37 PM · Feature Request
chad merged T11282: Clearly indicate in email footer how to unsubscribe from similar emails into T9412: Annotate all the reasons that a user is included on the recipient list for mail.
Jul 6 2016, 5:37 PM · llvm, Restricted Project, Mail
chad updated the description for M1458: Homepage 2016.
Jul 6 2016, 5:36 PM
swisspol created T11282: Clearly indicate in email footer how to unsubscribe from similar emails.
Jul 6 2016, 5:35 PM · Feature Request
chad updated the image for Bug Report from Unknown Object (File) to Unknown Object (File).
Jul 6 2016, 4:41 PM
chad updated the image for Phame from F824325: project.png to F1713938: profile.
Jul 6 2016, 4:40 PM
chad added a comment to T11281: Change language to spanish.

Phabricator by default doesn't include localization/translations.

Jul 6 2016, 4:05 PM · Feature Request
chad merged T11281: Change language to spanish into T5267: Localize Phabricator.
Jul 6 2016, 4:05 PM · Prioritized, Localization
chad merged task T11281: Change language to spanish into T5267: Localize Phabricator.
Jul 6 2016, 4:04 PM · Feature Request
yaznelm created T11281: Change language to spanish.
Jul 6 2016, 3:11 PM · Feature Request
epriestley added a comment to T11277: Provide a generic SSH public key store for users.

SSH keys only got half-upgraded to modern stuff in T10917, but we could finish that at some point and let you tag them with projects -- roughly the same as renaming them, but a little more formal.

Jul 6 2016, 2:42 PM · Auth, Feature Request
epriestley added a comment to T11280: Archiving/Activating Pastes reflects wrong in Recent Activity Log.

Thanks for the report! This should be resolved in HEAD of master.

Jul 6 2016, 2:21 PM · Paste, Bug Report
epriestley closed T11280: Archiving/Activating Pastes reflects wrong in Recent Activity Log as Resolved by committing Restricted Diffusion Commit.
Jul 6 2016, 2:17 PM · Paste, Bug Report
plehmkuhl added a comment to T10750: Cannot Edit Comment on Files.

We are also able to reproduce this on our installation.
Currently we are using Version 05699388804fc6ae1df5a17559917d218c4ec26d of phabricator.

Jul 6 2016, 11:15 AM · Files, Bug Report
epriestley added a revision to T11280: Archiving/Activating Pastes reflects wrong in Recent Activity Log: Unknown Object (Differential Revision).
Jul 6 2016, 11:00 AM · Paste, Bug Report
epriestley claimed T11280: Archiving/Activating Pastes reflects wrong in Recent Activity Log.
Jul 6 2016, 10:59 AM · Paste, Bug Report
chr86 created T11280: Archiving/Activating Pastes reflects wrong in Recent Activity Log.
Jul 6 2016, 9:29 AM · Paste, Bug Report
hach-que closed T11279: Phabricator daemon fails when updating repository (daemon status Running -> Unknown -> Dead ) as Invalid.

Your answer is right here:

Jul 6 2016, 7:07 AM · Bug Report
• leeray created T11279: Phabricator daemon fails when updating repository (daemon status Running -> Unknown -> Dead ).
Jul 6 2016, 6:38 AM · Bug Report
eadler merged T6297: Maniphest email notification doesn't say why I'm receiving it into T9412: Annotate all the reasons that a user is included on the recipient list for mail.
Jul 6 2016, 4:27 AM · llvm, Restricted Project, Mail
eadler merged task T6297: Maniphest email notification doesn't say why I'm receiving it into T9412: Annotate all the reasons that a user is included on the recipient list for mail.
Jul 6 2016, 4:27 AM · Maniphest
chad added a comment to T11278: It's confusing to figure out in the UI how to import a repository now.

yeah doesn't import ssh from what I see and they just say to use the command-line importer if you can't use the web tool

Jul 6 2016, 2:32 AM · Design, Diffusion
epriestley added a comment to T11278: It's confusing to figure out in the UI how to import a repository now.

Here's what I see -- step 1:

Jul 6 2016, 2:28 AM · Design, Diffusion
isfs added a revision to T10920: Allow assigning a diff to a commit post-facto: Unknown Object (Differential Revision).
Jul 6 2016, 2:11 AM · Diffusion, Feature Request
chad added a comment to T11278: It's confusing to figure out in the UI how to import a repository now.

I get this screen:

Jul 6 2016, 2:10 AM · Design, Diffusion
epriestley added a comment to T11278: It's confusing to figure out in the UI how to import a repository now.

GitHub's import doesn't seem to work with any repositories that require credentials?

Jul 6 2016, 2:04 AM · Design, Diffusion
epriestley added a comment to T11278: It's confusing to figure out in the UI how to import a repository now.

Oh, I was looking at this:

Jul 6 2016, 2:01 AM · Design, Diffusion
chad added a comment to T11278: It's confusing to figure out in the UI how to import a repository now.

pasted_file (534×590 px, 62 KB)

Jul 6 2016, 2:01 AM · Design, Diffusion
chad added a comment to T11278: It's confusing to figure out in the UI how to import a repository now.

pasted_file (159×286 px, 11 KB)

Jul 6 2016, 2:00 AM · Design, Diffusion
epriestley added a comment to T11278: It's confusing to figure out in the UI how to import a repository now.

GitHub make this reasonably obvious.

Jul 6 2016, 1:59 AM · Design, Diffusion
chad added a comment to T11278: It's confusing to figure out in the UI how to import a repository now.

People keep getting stuck on this, so we should look at fixing the UI so we don't have to post the documentation over and over.

Jul 6 2016, 1:52 AM · Design, Diffusion
chad renamed T11278: It's confusing to figure out in the UI how to import a repository now from Can no longer find 'import from existing repository' to It's confusing to figure out in the UI how to import a repository now.
Jul 6 2016, 1:51 AM · Design, Diffusion
• vanblaze added a comment to T11278: It's confusing to figure out in the UI how to import a repository now.

Thanks.

Jul 6 2016, 1:50 AM · Design, Diffusion
chad closed T11278: It's confusing to figure out in the UI how to import a repository now as Invalid.

https://secure.phabricator.com/book/phabricator/article/diffusion_uris/

Jul 6 2016, 1:47 AM · Design, Diffusion
• vanblaze created T11278: It's confusing to figure out in the UI how to import a repository now.
Jul 6 2016, 1:44 AM · Design, Diffusion
hach-que added a comment to T11277: Provide a generic SSH public key store for users.

After further investigation, we might just be able to tell users to set the "Name" field of the key that they want to use for production bastion access. I hadn't previously realised that this field was editable by clicking on the key in the "SSH Public Keys" pane.

Jul 6 2016, 12:36 AM · Auth, Feature Request
hach-que created T11277: Provide a generic SSH public key store for users.
Jul 6 2016, 12:33 AM · Auth, Feature Request

Jul 5 2016

epriestley closed T11275: Missing "Edit Queries..." and "Advanced Search" pages in User Settings as Resolved by committing Restricted Diffusion Commit.
Jul 5 2016, 11:55 PM · Bug Report
epriestley closed T11276: diffusion.uri.edit for creating an object fails depending on order of transactions in input as Resolved by committing Restricted Diffusion Commit.
Jul 5 2016, 11:45 PM · ApplicationEditor, Conduit, Bug Report
epriestley added a revision to T11275: Missing "Edit Queries..." and "Advanced Search" pages in User Settings: Unknown Object (Differential Revision).
Jul 5 2016, 11:45 PM · Bug Report
epriestley added a revision to T11276: diffusion.uri.edit for creating an object fails depending on order of transactions in input: Unknown Object (Differential Revision).
Jul 5 2016, 11:40 PM · ApplicationEditor, Conduit, Bug Report
epriestley added a comment to T11276: diffusion.uri.edit for creating an object fails depending on order of transactions in input.

A possible alternate approach would be to attachRepository() as a side-effect of validating the transaction instead of while applying the transaction, but then if something throws between the "validate" and "apply" steps we could end up with a URI in a weird state. That probably never actually matters, but feels a little more awkward to me than ordering the transactions.

Jul 5 2016, 11:34 PM · ApplicationEditor, Conduit, Bug Report
avivey added a comment to T11276: diffusion.uri.edit for creating an object fails depending on order of transactions in input.

ok, that makes sense.

Jul 5 2016, 11:32 PM · ApplicationEditor, Conduit, Bug Report
epriestley added a comment to T11276: diffusion.uri.edit for creating an object fails depending on order of transactions in input.

It's impossible to create a URI without a repository transaction (it doesn't make sense, since URI objects can't just exist), and we already give you an appropriate error when you try:

Jul 5 2016, 11:31 PM · ApplicationEditor, Conduit, Bug Report
avivey added a comment to T11276: diffusion.uri.edit for creating an object fails depending on order of transactions in input.

That kinda feels like a band-aid... Shouldn't we be able to not-crash for any input? Like if a user will only provide a uri transaction?

Jul 5 2016, 11:22 PM · ApplicationEditor, Conduit, Bug Report
epriestley added a comment to T9275: Move Calendar to EditEngine.

One problem with updating this is that recurring events currently work like this:

Jul 5 2016, 10:50 PM · ApplicationEditor, Policy, Calendar
epriestley added a comment to T11276: diffusion.uri.edit for creating an object fails depending on order of transactions in input.

My initial thought is that we should just reorder these transactions. We should use a stable sort, but that's relatively easy now that we have msortv(). There are probably a handful of other transactions with similar properties in other object types.

Jul 5 2016, 9:10 PM · ApplicationEditor, Conduit, Bug Report
avivey renamed T11276: diffusion.uri.edit for creating an object fails depending on order of transactions in input from diffusion.uri.edit fails depending on order of transactions in input to diffusion.uri.edit for creating an object fails depending on order of transactions in input.
Jul 5 2016, 8:43 PM · ApplicationEditor, Conduit, Bug Report
avivey created T11276: diffusion.uri.edit for creating an object fails depending on order of transactions in input.
Jul 5 2016, 8:42 PM · ApplicationEditor, Conduit, Bug Report
jack.chung added a comment to T10456: Conduit interprets all arguments from curl requests as strings.

if it helps anyone, the workaround "fix" is to place the following code in the file "src/applications/conduit/parametertype/ConduitBoolParameterType.php "

Jul 5 2016, 8:26 PM · Restricted Project, Conduit, Bug Report
avivey added a revision to T9789: Make it easier to write custom transaction types: Unknown Object (Differential Revision).
Jul 5 2016, 8:17 PM · Modernization, Transactions
aik099 created T11275: Missing "Edit Queries..." and "Advanced Search" pages in User Settings.
Jul 5 2016, 7:48 PM · Bug Report
epriestley added a revision to T9275: Move Calendar to EditEngine: Unknown Object (Differential Revision).
Jul 5 2016, 4:30 PM · ApplicationEditor, Policy, Calendar
epriestley claimed T9275: Move Calendar to EditEngine.
Jul 5 2016, 4:28 PM · ApplicationEditor, Policy, Calendar
epriestley added a comment to T11269: Repository update command fails on empty Git repositories.

Thanks for the report! I believe this should now be resolved in HEAD of master.

Jul 5 2016, 4:10 PM · Diffusion, Bug Report
epriestley closed T11269: Repository update command fails on empty Git repositories as Resolved by committing Restricted Diffusion Commit.
Jul 5 2016, 4:09 PM · Diffusion, Bug Report
epriestley added a comment to T10633: Render timezones explicitly in email.

T9854 is also vaguely related, but fairly separate.

Jul 5 2016, 4:06 PM · Calendar, Mail, Countdown, Bug Report
epriestley closed T7929: Integrate Calendar with external systems as Resolved.

I'm just going to close this in favor of the specific ICS (T10747) and Google Calendar (T2334) tasks, which are the only systems we plan to target for now and don't have much in common.

Jul 5 2016, 3:59 PM · Calendar
epriestley closed T9606: "Calendar" tab on user profiles is pretty junky as Resolved.

I think this is in reasonable shape, now.

Jul 5 2016, 3:58 PM · Calendar
epriestley closed T9606: "Calendar" tab on user profiles is pretty junky, a subtask of T7925: Calendar v1 Errata, as Resolved.
Jul 5 2016, 3:58 PM · Calendar
epriestley closed T10920: Allow assigning a diff to a commit post-facto as Resolved.

Yes, commit/revision and commit/task relationships can now be managed manually from the web UI.

Jul 5 2016, 3:52 PM · Diffusion, Feature Request
nochum added a comment to T10920: Allow assigning a diff to a commit post-facto.

I'm pretty sure this is fixed now with the Edit Related Objects menu awesomeness! Thanks.

Jul 5 2016, 3:32 PM · Diffusion, Feature Request
epriestley added a revision to T11269: Repository update command fails on empty Git repositories: Unknown Object (Differential Revision).
Jul 5 2016, 3:30 PM · Diffusion, Bug Report
epriestley claimed T11269: Repository update command fails on empty Git repositories.
Jul 5 2016, 3:26 PM · Diffusion, Bug Report
epriestley merged T11273: Update error while Repository Updates into T11269: Repository update command fails on empty Git repositories.
Jul 5 2016, 3:26 PM · Diffusion, Bug Report
epriestley merged task T11273: Update error while Repository Updates into T11269: Repository update command fails on empty Git repositories.
Jul 5 2016, 3:26 PM · Bug Report
epriestley added a comment to T11273: Update error while Repository Updates .

I'm going to merge this into T11269, which I believe describes the same problem.

Jul 5 2016, 3:25 PM · Bug Report
epriestley added a comment to T11274: Task Graph has extra vertical space on tickets with titles that run onto multiple lines.

Thanks for the report! I think this should be fixed at HEAD of master. Let us know if you're still seeing issues after updating.

Jul 5 2016, 3:11 PM · Bug Report
epriestley closed T11274: Task Graph has extra vertical space on tickets with titles that run onto multiple lines as Resolved by committing Restricted Diffusion Commit.
Jul 5 2016, 3:00 PM · Bug Report
aubort added a comment to T11273: Update error while Repository Updates .

I'm having the same issue since I updated from week 21 to week 27, it happens on empty repositories.

Jul 5 2016, 2:54 PM · Bug Report
vpetersson added a comment to T4721: Improve user-facing documentation for Passphrase.

And we should possibly consider splitting the permissions more granularly:

  • View
  • Show Secret
  • Edit

+1

Jul 5 2016, 2:31 PM · Documentation, Passphrase
epriestley claimed T11274: Task Graph has extra vertical space on tickets with titles that run onto multiple lines.
Jul 5 2016, 1:55 PM · Bug Report
epriestley added a revision to T11274: Task Graph has extra vertical space on tickets with titles that run onto multiple lines: Unknown Object (Differential Revision).
Jul 5 2016, 1:54 PM · Bug Report
trevorsummerssmith created T11274: Task Graph has extra vertical space on tickets with titles that run onto multiple lines.
Jul 5 2016, 1:46 PM · Bug Report
epriestley added a comment to T10745: Emails are sent out about restricted changes.

The support cost for the current behavior is very small. The behavior of the system is obvious. Users can file precise tasks like this one which clearly describe the problem (or find this task):

Jul 5 2016, 1:45 PM · Restricted Project, Bug Report
• Maxim moved T3569: Make Phrequent somewhat usable from Backlog to middlelog on the Phrequent board.
Jul 5 2016, 1:15 PM · Phrequent
• Maxim moved T5151: Allow to export query results from Phrequent from Backlog to middlelog on the Phrequent board.
Jul 5 2016, 1:15 PM · Phrequent
• Maxim moved T2928: "arc start" / "arc feature" / "arc push" Discussion from Backlog to middlelog on the Phrequent board.
Jul 5 2016, 1:15 PM · Phrequent, Maniphest, Arcanist
• Maxim moved T3999: Support Harvest integration for Phrequent via Doorkeeper from Backlog to middlelog on the Phrequent board.
Jul 5 2016, 1:15 PM · Phrequent, Doorkeeper
• Maxim moved T5149: Allow to add and edit Phrequent entries manually from Backlog to middlelog on the Phrequent board.
Jul 5 2016, 1:15 PM · Phrequent
• Maxim moved T8480: Support querying for tasks where a user has tracked time against them from Backlog to middlelog on the Phrequent board.
Jul 5 2016, 1:14 PM · Phrequent, Maniphest
• Maxim moved T5150: Extend query by time restrictions from Backlog to middlelog on the Phrequent board.
Jul 5 2016, 1:14 PM · Phrequent
• Maxim moved T8666: When a task is resolved with "Resolves T123", Phrequent should stop tracking time from Backlog to middlelog on the Phrequent board.
Jul 5 2016, 1:14 PM · Maniphest, Phrequent
epriestley added a project to T4706: Maybe make `arc land D123` go look up the branch?: Prioritized.
Jul 5 2016, 12:37 PM · Prioritized, Restricted Project, Arcanist
epriestley added a project to T10895: Support`arc browse --revision <commit>` and make `arc browse` with no arguments mean `... --revision HEAD`: Prioritized.
Jul 5 2016, 12:37 PM · Arcanist, Prioritized, Restricted Project, Feature Request
epriestley merged T11272: Passphrases requires Editable By status for Read access into T4721: Improve user-facing documentation for Passphrase.
Jul 5 2016, 12:36 PM · Documentation, Passphrase
epriestley merged task T11272: Passphrases requires Editable By status for Read access into T4721: Improve user-facing documentation for Passphrase.
Jul 5 2016, 12:36 PM · Bug Report
epriestley added a comment to T11272: Passphrases requires Editable By status for Read access.

This behavior is intentional; it allows users to use a credential to configure other applications without actually having access to the plaintext.

Jul 5 2016, 12:36 PM · Bug Report
epriestley closed T9888: Provide guidance in the documentation about PHP7, a subtask of T9640: Make Phabricator compatible with PHP7, as Resolved.
Jul 5 2016, 12:33 PM · Prioritized, Restricted Project, Infrastructure
epriestley closed T9888: Provide guidance in the documentation about PHP7 as Resolved by committing Restricted Diffusion Commit.
Jul 5 2016, 12:33 PM · Documentation
epriestley closed T11250: New Herald rule: "Differential revision accepted by" as Resolved.

Great! I think the Herald rule is also reasonable to build down the line if some future user arrives here via search and has a different use case for it in the future.

Jul 5 2016, 11:44 AM · Feature Request
• minishine updated the task description for T11273: Update error while Repository Updates .
Jul 5 2016, 11:34 AM · Bug Report
• minishine updated the task description for T11273: Update error while Repository Updates .
Jul 5 2016, 11:28 AM · Bug Report
• minishine updated the task description for T11273: Update error while Repository Updates .
Jul 5 2016, 11:28 AM · Bug Report
• minishine created T11273: Update error while Repository Updates .
Jul 5 2016, 11:25 AM · Bug Report
vpetersson created T11272: Passphrases requires Editable By status for Read access.
Jul 5 2016, 11:24 AM · Bug Report
vit0p updated Q439: There is a conduit method to retrieve differentials attached to a maniphest task? from to There is a conduit method to retrieve differentials attached to a maniphest task?.
Jul 5 2016, 10:51 AM · Differential, Maniphest, Conduit
• emtized awarded T11269: Repository update command fails on empty Git repositories a Like token.
Jul 5 2016, 9:52 AM · Diffusion, Bug Report