FWIW, the API seems not intuitive for me: why does the user is able to set arbitrary properties via Conduit, if there is no way to get them back? Is exposing the API actually a hack, to achieve other goals?
Feed Advanced Search
Dec 13 2016
Dec 13 2016
epriestley renamed T12011: Support builds with Travis CI from There is no way to read diffproperty using Conduit to Support builds with Travis CI.
epriestley added a comment to T12011: Support builds with Travis CI.
epriestley added a revision to T11114: Move Differential to EditEngine: Unknown Object (Differential Revision).
mwek updated the task description for T12011: Support builds with Travis CI.
rfreebern added a comment to T12012: Diffusion doesn't handle non-bare hosted git repos.
Okay, thanks!
Oh, I think the issue is this:
epriestley added a comment to T12012: Diffusion doesn't handle non-bare hosted git repos.
I also can't reproduce this:
rfreebern added a comment to T12012: Diffusion doesn't handle non-bare hosted git repos.
It's fine to consider it not a bug. Mainly I wanted to provide the info that it doesn't work for documentation purposes, in case anyone else runs into the same situation in the future. (We ran into this by trying to transition from a very old Phabricator installation to a new one with hosted repos, but with a legacy deployment setup that needs a non-bare repo copy, and didn't realize that Phabricator didn't fully support hosted non-bare repos.)
epriestley added a comment to T12012: Diffusion doesn't handle non-bare hosted git repos.
Why do you consider this to be a bug, then?
epriestley added a comment to T1508: When running "arc diff" on a range of commits, preserve the individual commits while still creating a single diff from them.
Do you consider this ticket just "make arc patch restore all local commits", or something more?
rfreebern added a comment to T12012: Diffusion doesn't handle non-bare hosted git repos.
No.
avivey added a comment to T1508: When running "arc diff" on a range of commits, preserve the individual commits while still creating a single diff from them.
Now I'm even more confused. This ticket still doesn't conform to modern Feature Requests rules :)
epriestley renamed T1508: When running "arc diff" on a range of commits, preserve the individual commits while still creating a single diff from them from Allow arc diff --create" to create a diff/revision out of each local commit to When running "arc diff" on a range of commits, preserve the individual commits while still creating a single diff from them.
epriestley added a comment to T1508: When running "arc diff" on a range of commits, preserve the individual commits while still creating a single diff from them.
Oh, that's actually a different task, I don't currently plan to implement that. Let me me rename this to be more clear...
avivey renamed T1508: When running "arc diff" on a range of commits, preserve the individual commits while still creating a single diff from them from Upload local commits in "arc diff --create" to Allow arc diff --create" to create a diff/revision out of each local commit.
epriestley added a comment to T12012: Diffusion doesn't handle non-bare hosted git repos.
Do we tell you to do this in the documentation anywhere?
epriestley added a revision to T10967: Move Differential reviewers back to a dedicated database storage table: Unknown Object (Differential Revision).
epriestley added a revision to T8475: Clean up remnants of old Differential hunk storage: Unknown Object (Differential Revision).
epriestley added a revision to T8475: Clean up remnants of old Differential hunk storage: Unknown Object (Differential Revision).
epriestley updated the task description for T8623: Upgrading: Differential Hunk Migration.
epriestley closed T4045: Store diffs as binary, not UTF-8, a subtask of T1191: Comments are truncated at first non-base-plane character, as Resolved.
This is now better tracked in followups:
epriestley closed T4045: Store diffs as binary, not UTF-8, a subtask of T5229: Fixing up invalid encoding confuses archanists utf8-check, as Resolved.
epriestley closed T4045: Store diffs as binary, not UTF-8, a subtask of T5179: Unearth Differential rendering technical debt, as Resolved.
epriestley edited projects for T12011: Support builds with Travis CI, added: Feature Request (Needs Information); removed Bug Report.
epriestley added a comment to T12011: Support builds with Travis CI.
This isn't a bug. Diff properties are not intended to be an arbitrary key-value datastore for API consumers.
epriestley added a revision to T7899: DifferentialDiff->description is "text255?" but should be "text": Unknown Object (Differential Revision).
epriestley added a comment to T7899: DifferentialDiff->description is "text255?" but should be "text".
This field is only populated when you update a revision, and always contains the first line of the update message, which is also posted as a comment.
• rbtaw moved T5375: Tokenizers fail in Firefox on Android from Icebox to Backlog on the Android board.
chad awarded T12010: Untangle the Gordian Knot of iterating on Differential/Diffusion/Arcanist a Haypence token.
cspeckmim awarded T7899: DifferentialDiff->description is "text255?" but should be "text" a Dat Boi token.
epriestley moved T11057: Allow installs to manage which cluster service new repositories allocate on from Backlog to Clusters on the Diffusion board.
epriestley moved T6722: Support "Create similar repository..." in Diffusion from Backlog to v3 on the Diffusion board.
epriestley moved T6008: Editing files and contributing changes via web from Backlog to Far Future on the Diffusion board.
epriestley moved T929: Allow two arbitrary revisions to be compared in Diffusion from Backlog to v3 on the Diffusion board.
epriestley moved T10883: Allow repository cluster nodes to be read-only from Backlog to Clusters on the Diffusion board.
epriestley moved T5000: Using Differential with plain Git, without requiring Arc from Future Work to v3 (Infrastructure) on the Differential board.
epriestley moved T182: Commit into repository directly from differential from Future Work to v3 (Infrastructure) on the Differential board.
epriestley moved T731: Allow revisions to have alternate acceptance conditions from Backlog to v3 (Infrastructure) on the Differential board.
epriestley moved T2543: Add a formal "Draft" / "Not Yet Ready for Review" state to Differential from Backlog to v3 (Infrastructure) on the Differential board.
Dec 13 2016, 4:29 PM · Customer Impact, Restricted Project, Restricted Project, Prioritized, Differential
epriestley moved T4631: Allow Differential to raise warnings on the server side via Conduit from Backlog to v3 (Infrastructure) on the Differential board.
epriestley moved T8475: Clean up remnants of old Differential hunk storage from Backlog to v3 (Infrastructure) on the Differential board.
epriestley moved T8623: Upgrading: Differential Hunk Migration from Backlog to v3 (Infrastructure) on the Differential board.
This hasn't moved forward in a year so I'm just going to close it. We've removed header text in the latest UI iteration, too, and I imagine the button will be replaced completely with universal send-on-enter before long.
epriestley moved T10574: A pathway toward "All Reviewers Must Accept" from Backlog to v3 (Infrastructure) on the Differential board.
epriestley moved T10967: Move Differential reviewers back to a dedicated database storage table from Backlog to v3 (Infrastructure) on the Differential board.
epriestley moved T11050: Allow resigning from a review you are not directly responsible for from Backlog to v3 (Infrastructure) on the Differential board.
epriestley moved T11114: Move Differential to EditEngine from Backlog to v3 (Infrastructure) on the Differential board.
cspeckmim added a comment to T11708: Daemons erroneously saying "Diff does not exist".
When this is in stable and we upgrade I will have a reminder to keep an eye on logs and come back here
epriestley moved T4045: Store diffs as binary, not UTF-8 from v2 (UI/Mobile) to v3 (Infrastructure) on the Differential board.
epriestley moved T11660: Diff creation fails when submitting too long file paths from Backlog to v3 (Infrastructure) on the Differential board.
epriestley moved T11767: Unify email patch byte/line/time limit behaviors across Differential/Diffusion from Backlog to Future Work on the Differential board.
epriestley added a comment to T11708: Daemons erroneously saying "Diff does not exist".
Although it's still a little early to be sure, we appear to have fixed the policy issue with unit test failures (D17029). Not sure if that actually impacted this or not.
epriestley closed T12001: Documentation is misleading about the unit that "storage.mysql-engine.max-size" is measured in as Resolved by committing Restricted Diffusion Commit.
epriestley updated the task description for T12004: Scenario: Locations Application.
claudiof awarded T4280: Embed anchor in remark up text and later link to it. a Like token.
stevex awarded T5954: Modularize "Export to Excel" as a feature of ApplicationSearch a Like token.
epriestley closed T7457: On dashboards, maybe only draw panels if all panels above them have already drawn as Wontfix.
My dashboard changed at some point and no one else seems to care, so let's wait for this to be more than an @epriestley-problem.
epriestley renamed T12001: Documentation is misleading about the unit that "storage.mysql-engine.max-size" is measured in from Upload Failure to Documentation is misleading about the unit that "storage.mysql-engine.max-size" is measured in.
epriestley added a comment to T12001: Documentation is misleading about the unit that "storage.mysql-engine.max-size" is measured in.
D17037 should fix the documentation, it definitely wasn't very clear as written. That should land some time today and the docs should be updated in about 24 hours.
Darakir added a comment to T12001: Documentation is misleading about the unit that "storage.mysql-engine.max-size" is measured in.
Omg. I feel so stupid now. In the guide I read, that should be interpreted as 8MB, but I guess it wasn't... Works fine now, thanks a bunch!
This can be closed..
epriestley added a revision to T12001: Documentation is misleading about the unit that "storage.mysql-engine.max-size" is measured in: Unknown Object (Differential Revision).
epriestley added a comment to T12001: Documentation is misleading about the unit that "storage.mysql-engine.max-size" is measured in.
In the first row of that table, it appears that you have configured a storage.mysql-engine.max-size of 8 ("8 bytes"). Try using 8388608 ("8 megabytes") instead?
pouyana updated the task description for T12002: Set header for CURL query for the Notification server status to HTTP:Upgrade = websocket.
If you want to come up with a specific list of domains I'm happy to buy them, but there are many hundreds of domains which differ from phacility.com by only one keystroke and buying hundreds of domains isn't cheap in terms of the amount of work I have to do to do it.
Darakir added a comment to T12001: Documentation is misleading about the unit that "storage.mysql-engine.max-size" is measured in.
Sure, here goes:
epriestley added a comment to T12003: Explain to users how fulltext queries are parsed and executed.
I can find this task by searching for "あああああ" so the CJK stuff may not really be much of an issue anymore? Maybe just an issue with Chinese because the index doesn't know how to tokenize words?
epriestley closed T6841: Maybe create an annotation for "this is strictly a template class" as Wontfix.
At least today, our error rate on this seems so low that it probably isn't worth implementing. We could take another look at it if we see more of it in the future (perhaps after we begin encouraging application development), but we haven't seen more of this that I can recall recently.
epriestley renamed T6633: 'differential.creatediff' exception when repository paths contain SHIFT-JIS characters from 'differential.creatediff' exception to 'differential.creatediff' exception when repository paths contain SHIFT-JIS characters.
Paste now shows "Old", "New" and "Diff", so you can copy/paste from "Old" to revert. Not perfect, but I think this arises relatively rarely. See T11881 for a followup.
epriestley closed T6592: Differential history mangles bullet numbering in quoted email forward as Resolved.
This is extremely old and it's not clear to me which bad cases remain -- I can't immediately find any by fiddling. If any do remain, they haven't turned up again in about two years. If you're still seeing issues, feel free to file a new task with a modern test case.
richardvanvelzen added a comment to T6389: Ship TShirt.
No worries, totally understandable.
This picked up a few options through engine support and T11957 is adding some more. Particularly, default view is now "Authored" which I think resolves the original issue. I think the more-root problem is "Dashboards are kind of weird with all the panel copying", which is something we're aware of.
I am very sorry I am declaring t-shirt bankruptcy because I have learned it takes me over five hundred years to ship a t-shirt.
d.maznekov updated the question details for Q535: Can be changed Default Space ?.
epriestley added a comment to T11957: Add MenuEngine to Home.
At some point after we get this working for Home, I want to replace the "Quick Create" menu with a similar customizable profile menu, resolving T5867.
epriestley added a subtask for T11957: Add MenuEngine to Home: T5867: Move Quick Create to MenuItemEngine.
epriestley added a parent task for T5867: Move Quick Create to MenuItemEngine: T11957: Add MenuEngine to Home.
epriestley renamed T6188: User should receive a nicer exception when selecting an invalid application policy like "Can Use Application: No One" from "Can Use Application" policy value "No One" useless? to User should receive a nicer exception when selecting an invalid application policy like "Can Use Application: No One".