cspeckmim (Christopher Speck)
Xevioid

Projects

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Saturday

  • Clear sailing ahead.

User Details

User Since
Dec 8 2014, 6:48 PM (120 w, 2 d)
Availability
Available

Software Engineering Lead

MIM Software, Inc.

http://www.mimsoftware.com/

{F1081352}

Recent Activity

Mon, Mar 27

cspeckmim added a comment to T9900: Figlet does not properly render apostrophes with some characters.

is srs bsness mode taking over? i'm in a panic :[

Mon, Mar 27, 4:44 PM · Contributor Onboarding, Remarkup
cspeckmim added a comment to T11641: The at-mention of a disabled user renders a line instead of a dot for inline comments.

high priority!

Mon, Mar 27, 3:56 PM · Restricted Project, Remarkup, Bug Report
cspeckmim created T12453: Using `arc patch` will leave behind a bookmark when answering 'No' in attempt to apply to current state in Mercurial repository.
Mon, Mar 27, 1:33 PM · Arcanist, Mercurial, Bug Report

Sun, Mar 26

cspeckmim created P2033 Example diff generated from mercurial.
Sun, Mar 26, 8:19 PM

Fri, Mar 24

cspeckmim updated the language for P2032 Creating diff from python using Conduit from remarkup to python.
Fri, Mar 24, 2:11 PM
cspeckmim created P2032 Creating diff from python using Conduit.
Fri, Mar 24, 2:11 PM

Thu, Mar 23

cspeckmim added a comment to D17541: Don't show the "Override Lock" prompt when creating objects.

FWIW I patched this locally and it fixes the scenarios I've seen this issue.

Thu, Mar 23, 3:01 PM
cspeckmim added a comment to T12369: Users sometimes prompted to "Edit Locked Task" when attempting to create a new task.

After updating the task a million times I did reduce the reproduction steps to just not having the user in the Default View Policy (presumably at time of creation the creator is not in the Subscribers policy group explaining the others' scenarios. I had updated the description to note that but I chould've made that much more clear. Spaces were totally a red herring.

Thu, Mar 23, 1:59 PM · Bug Report, EditEngine
cspeckmim added a comment to T12441: Implement "Phabricator Stories".

@phillco - Is this for like, seeing code reviews that haven't had active participation in while? So users could see what's not being reviewed and lend a hand? Under the assumption that the "most-qualified" stakeholders (or Owners?) are already reviewers/subscribers of a code change but aren't being active, how effective would others' review be?

Thu, Mar 23, 12:24 AM

Wed, Mar 22

cspeckmim accepted D17530: Remove duplicate "Change Default Values" action in form editing workflow.

:D

Wed, Mar 22, 3:56 PM
cspeckmim edited the description of T12369: Users sometimes prompted to "Edit Locked Task" when attempting to create a new task.
Wed, Mar 22, 3:55 AM · Bug Report, EditEngine
cspeckmim edited projects for T12369: Users sometimes prompted to "Edit Locked Task" when attempting to create a new task, added: Bug Report; removed Bug Report (Needs Information).
Wed, Mar 22, 3:46 AM · Bug Report, EditEngine
cspeckmim edited the description of T12369: Users sometimes prompted to "Edit Locked Task" when attempting to create a new task.
Wed, Mar 22, 3:45 AM · Bug Report, EditEngine
cspeckmim edited the description of T12369: Users sometimes prompted to "Edit Locked Task" when attempting to create a new task.
Wed, Mar 22, 3:30 AM · Bug Report, EditEngine
cspeckmim edited the description of T12369: Users sometimes prompted to "Edit Locked Task" when attempting to create a new task.
Wed, Mar 22, 3:25 AM · Bug Report, EditEngine
cspeckmim edited the description of T12369: Users sometimes prompted to "Edit Locked Task" when attempting to create a new task.
Wed, Mar 22, 3:24 AM · Bug Report, EditEngine
cspeckmim renamed T12369: Users sometimes prompted to "Edit Locked Task" when attempting to create a new task from "Locked task on create new one" to "Users sometimes prompted to "Edit Locked Task" when attempting to create a new task".
Wed, Mar 22, 3:23 AM · Bug Report, EditEngine
cspeckmim added a comment to T12369: Users sometimes prompted to "Edit Locked Task" when attempting to create a new task.

Thanks @chad for the help and motivation~

Wed, Mar 22, 3:18 AM · Bug Report, EditEngine
cspeckmim added a comment to T12369: Users sometimes prompted to "Edit Locked Task" when attempting to create a new task.

The key ingredient is to next set the default View/Edit policy of Maniphest to the Employees group (for Default Space which user is not accessible to)

Wed, Mar 22, 3:16 AM · Bug Report, EditEngine
cspeckmim added a comment to T12369: Users sometimes prompted to "Edit Locked Task" when attempting to create a new task.

what's your policy on supporting "random" PHP snippets from "sources" into Phabricator code

Wed, Mar 22, 2:55 AM · Bug Report, EditEngine
cspeckmim added a comment to T12369: Users sometimes prompted to "Edit Locked Task" when attempting to create a new task.

i have hosed it?

Wed, Mar 22, 2:52 AM · Bug Report, EditEngine
cspeckmim added a comment to T12369: Users sometimes prompted to "Edit Locked Task" when attempting to create a new task.

What is the best way to log something to the web server logs? I've been mostly using exceptions but that's proving to be more and more cumbersome.

Wed, Mar 22, 2:48 AM · Bug Report, EditEngine
cspeckmim added a comment to T12369: Users sometimes prompted to "Edit Locked Task" when attempting to create a new task.

I tried reproducing this on my phacility instance but I couldn't. Additional steps I took to mimic my setup were:

  1. Make the Default Space only accessible to a specific group (say, Employees)
  2. Make the default Maniphest Task Form visible only to Employees
  3. Make the new Contractors task form default to being in the {Contractors} Space, and only visible/editable by Contractors
Wed, Mar 22, 2:05 AM · Bug Report, EditEngine
cspeckmim added a comment to T12434: Editing Maniphest's EditEngine form shows "Change Default Values" twice.

Maybe this should've been reported on T10004?

Wed, Mar 22, 1:57 AM · EditEngine, Bug Report
cspeckmim created T12434: Editing Maniphest's EditEngine form shows "Change Default Values" twice.
Wed, Mar 22, 1:54 AM · EditEngine, Bug Report
cspeckmim added a comment to T12369: Users sometimes prompted to "Edit Locked Task" when attempting to create a new task.

I'm also running last week's install
libphutil: 24ede7a5dbfd38079c87fc61de64012551965837
arcanist: 822bc53ca306e06314560d8a76f68771d732e8e0
phabricator: 56dd1b297c3e5cdbb477acc7435d6aa5749f33f2

Wed, Mar 22, 1:05 AM · Bug Report, EditEngine
cspeckmim added a comment to T12369: Users sometimes prompted to "Edit Locked Task" when attempting to create a new task.

I am seeing this too. I've not confirmed 100% but I think it has to do with Spaces:

  1. Create a Space (not the default Space) visible to Project Contractors
  2. Create a Maniphest Task Form (/transactions/editengine/maniphest.task/) that is visible to Contractors, set as being Edit Form
  3. Create a User that is only a member of Contractors and which does not have access to the default space
  4. Log in as the new user and attempt to create a new task using that task form. You will be presented with the "Edit Locked Task" form.
Wed, Mar 22, 1:03 AM · Bug Report, EditEngine

Thu, Mar 16

cspeckmim added a comment to T12408: Security: "Show Raw File" in Differential generated files with overbroad permissions.

We ran the script provided above to get an audit of at-risk files. Afterwards we upgraded our instance and the upgrade succeeded however its attempts to delete the affected files failed. The failure is due to using a local file store which is accessible to our web service account but not the phabricator phd services account (T4752). After correcting the file permissions so both accounts have appropriate access, running upgrade again doesn't seem to remove the files.

Thu, Mar 16, 8:11 PM · Differential, Files, Security

Tue, Mar 14

cspeckmim added Q585: Using Spaces within a large organisation (Answer 530).

This guide should have most of the information you ask about:
https://secure.phabricator.com/book/phabricator/article/spaces/

Tue, Mar 14, 9:42 PM

Sat, Mar 11

cspeckmim added a comment to T11892: Aphlict does not work using SSL and node 6.9.1.

Tried again with node v6.9.4 (current version of node with CentOS7), and still has same problem 😢

Sat, Mar 11, 8:07 PM · Aphlict, Bug Report

Fri, Mar 10

cspeckmim edited the description of T12383: Accumulate Phabricator changelogs applicable from a current install.
Fri, Mar 10, 9:55 PM · Feature Request, Installing & Upgrading
cspeckmim added a comment to T12383: Accumulate Phabricator changelogs applicable from a current install.

That's a good point about also accumulating any security notes

Fri, Mar 10, 9:51 PM · Feature Request, Installing & Upgrading
cspeckmim updated subscribers of T12383: Accumulate Phabricator changelogs applicable from a current install.

Asking someone who is say, 16 months behind to read 80 changelogs seems detrimental to having people keep up to date.

This is partly why I would like this -- the Upgrade / Compatibility section of changelogs is probably the only near-mandatory sections to review before upgrading (things might behave differently until you do databasey things, the way a configuration behaves has changed, etc.). My process for upgrading right now begins with reviewing the changelogs to make sure I don't miss any pre/post-intall steps, and I'd like to automate part of that if possible. For a site that's upgrading weekly this isn't a challenge, but sites that are unable to upgrade as frequently would (should?) be sifting through this stuff anyways. I think this would be part of community resources and not officially part of Phabricator.

Fri, Mar 10, 5:35 PM · Feature Request, Installing & Upgrading
cspeckmim added a comment to T12383: Accumulate Phabricator changelogs applicable from a current install.

Did I really have an extra space in the title

Fri, Mar 10, 5:21 PM · Feature Request, Installing & Upgrading
cspeckmim created T12383: Accumulate Phabricator changelogs applicable from a current install.
Fri, Mar 10, 5:09 PM · Feature Request, Installing & Upgrading
cspeckmim added a comment to T12380: Errors with Mercurial trust while browsing rHGTEST on this install.

Different parts of the repository page may or may not load indicating error. I refreshed on the page with the commit a few times and sometimes it says the commit is missing and other times it can't load the content, and other times it can't load the paths, but sometimes it can load them.

Fri, Mar 10, 5:32 AM · Mercurial, Bug Report
cspeckmim created T12380: Errors with Mercurial trust while browsing rHGTEST on this install.
Fri, Mar 10, 5:28 AM · Mercurial, Bug Report

Thu, Mar 9

cspeckmim added a comment to T12370: Maniphest menu items misalign if they have submenu items.

youreawesome

Thu, Mar 9, 3:27 PM · Bug Report

Wed, Mar 8

cspeckmim added a comment to T9430: "Needs Audit" no longer excludes author commits.

I'm good with the current balance of the changelogs - the one thing I think would be nifty is being able to accumulate the Upgrade/Compatibility sections across changelogs, for sites that aren't able to upgrade weekly. I was considering making a script to scrape that info but haven't had a chance to sit down for it. Also it'd be relying on the changelog format to be consistent (seems to be pretty darn consistent).

Wed, Mar 8, 10:59 PM · Restricted Project, Restricted Project, Audit
cspeckmim added a comment to T12370: Maniphest menu items misalign if they have submenu items.

I tried removing white space: nowrap; (from one of .phabricator-action-view button.phabricator-action-view-item, .phabricator-action-view-item) in the browser inspector on FF and it looks like it goes back to normal (even when expanding).

Wed, Mar 8, 8:52 PM · Bug Report
cspeckmim edited the description of T12370: Maniphest menu items misalign if they have submenu items.
Wed, Mar 8, 8:43 PM · Bug Report
cspeckmim edited the description of T12370: Maniphest menu items misalign if they have submenu items.
Wed, Mar 8, 8:42 PM · Bug Report
cspeckmim edited the description of T12370: Maniphest menu items misalign if they have submenu items.
Wed, Mar 8, 8:42 PM · Bug Report
cspeckmim created T12370: Maniphest menu items misalign if they have submenu items.
Wed, Mar 8, 8:41 PM · Bug Report

Fri, Mar 3

cspeckmim renamed T12344: Relative links to Phriction pages misinterprets URL-encoded characters from "Relative links to Phriction pages mis-encode URL-encoded characters" to "Relative links to Phriction pages misinterprets URL-encoded characters".
Fri, Mar 3, 3:15 AM · Phriction, Remarkup, Bug Report
cspeckmim created T12344: Relative links to Phriction pages misinterprets URL-encoded characters.
Fri, Mar 3, 3:13 AM · Phriction, Remarkup, Bug Report
cspeckmim added a comment to T12343: Link-Parsing fails if URL ends in ) when using "[name](link)" syntax.

Workaround is to URL-encode the parenthesis (%29) or use the remarkup style

[Back to the Future](https://en.wikipedia.org/wiki/Starship_Troopers_(film%29)
[[ https://en.wikipedia.org/wiki/Starship_Troopers_(film) | Jurassic Park ]]

Results in:
Back to the Future
Jurassic Park

Fri, Mar 3, 2:58 AM · Remarkup, Bug Report
cspeckmim added a comment to T12335: Ability to lock a task to prevent further edits.

One use case for "locking" a task could be:

  1. For a release of the software product all tasks which represent the software changes on the version being released require review
  2. Reviewer looks over task form fields and if everything looks good, "signs" (locks) the task in a "reviewed" state
  3. Tasks can be unsigned prior to release of the version, allowing updates to tasks as necessary but also requiring another review
  4. Upon release of the version, tasks can no longer be unsigned
Fri, Mar 3, 2:38 AM · Maniphest, Feature Request

Thu, Mar 2

cspeckmim added a comment to T12324: Dashboards that contain a restricted panel have all query panels stuck in 'Loading...'.

I like the idea of a feed/panel that shows recently active objects and not the individual events, but I don't know that I have a specific use case for it now. For how I use feed I think being able to filter out commits would be a nifty plus, as those tend to pollute the feed during arc land activity (and comments in tow). I don't think I feel too strongly about it right now though, as most of my work is driven via email activity and I primarily use Feed as a means of discoverability so I can poke my head in if something sounds interesting~

Thu, Mar 2, 5:49 AM · Dashboards, Bug Report

Wed, Mar 1

cspeckmim added a comment to T12174: Home Menu Errata.

Add catfacts tooltip to each application?

Wed, Mar 1, 3:38 PM · Dashboards (v2), Profile Menu Item

Feb 27 2017

cspeckmim added a comment to D17422: Add a lipsum generator for Badges.

Feb 27 2017, 2:00 PM

Feb 24 2017

cspeckmim added a comment to D17410: Build a Badges page for Profiles.

Not a big deal since it'll be a while before any install has enough badges for it to matter.

partycat

Feb 24 2017, 9:26 PM

Feb 23 2017

cspeckmim edited the description of T12309: Differential comment that inlines a Paste and has code block after it renders in strange fashion.
Feb 23 2017, 2:22 AM · Remarkup, Bug Report
cspeckmim closed T12309: Differential comment that inlines a Paste and has code block after it renders in strange fashion as "Invalid".

Ooh just kidding. Ctrl+Refresh fixed everything

Feb 23 2017, 2:21 AM · Remarkup, Bug Report
cspeckmim created T12309: Differential comment that inlines a Paste and has code block after it renders in strange fashion.
Feb 23 2017, 2:20 AM · Remarkup, Bug Report

Feb 22 2017

cspeckmim added a comment to T12308: Access forbidden on homepage if the first dashboard in menu is not visible to user.

In my scenario the external companies would not see eachother's tasks (separate Spaces), nor other internal items (default Space). However the developers within my company would need to see things from across multiple Spaces.

Feb 22 2017, 4:30 PM · Dashboards, Profile Menu Item, Bug Report
cspeckmim added a comment to T12308: Access forbidden on homepage if the first dashboard in menu is not visible to user.

Do these dashboards contain sensitive information?

For the use-case I was trying to outline in T9501, this would be the case. We work with a handful of external companies and use Phabricator for organizing and communicating work. We also use Phabricator for our code review. Each project with external company (People, Projects, Maniphest) is organized into separate Spaces so that they are fully isolated, and we have a default space for internal use for Diffusion/Differential. Ideally when a user from Company A logs in they would land on a dashboard that shows only the Projects/Maniphest for them, while Company B user logs in and only shows the Projects/Maniphest for them.

Feb 22 2017, 3:42 PM · Dashboards, Profile Menu Item, Bug Report
cspeckmim added a comment to T12308: Access forbidden on homepage if the first dashboard in menu is not visible to user.

Probably very similar or even dupe of T9501

Feb 22 2017, 2:28 PM · Dashboards, Profile Menu Item, Bug Report

Feb 21 2017

cspeckmim added a comment to T11708: Daemons erroneously saying "Diff does not exist".

I don't believe we've seen this pop up again. I can't be certain because right now all the log files are empty, but that means the errors went away?

Feb 21 2017, 5:26 AM · Daemons, Differential, Bug Report

Feb 20 2017

cspeckmim added a comment to T12299: Linking Maniphest Tasks to external Bugtrackers.

You can make your own extension to Remarkup to allow auto-linking. I have one setup which I based off of P1129, P1209 which regex matches on Task-xxxx and uses the xxxx to link to our system. That works well and is sufficient for what we need.

Feb 20 2017, 5:29 PM · Feature Request

Feb 19 2017

cspeckmim added a comment to T12174: Home Menu Errata.

Just tested this out with having global menu items of Create Task forms and Projects, each which are only visible based on some policy - looks to not have any issues hiding/showing based on those policies. Thanks for all this, when I get these changes into production install I'll check for feedback but I'm anticipating it will be a smoother experience for everyone.

Feb 19 2017, 5:18 PM · Dashboards (v2), Profile Menu Item
cspeckmim added Q570: How to install `Phrequent`? (Answer 514).

Phrequent is a prototype application which is not supported. If you want to try it out understanding this, you must change a configuration allowing prototype applications (see config phabricator.show-prototypes), then you will be able to install the application.

Feb 19 2017, 3:41 PM

Feb 18 2017

cspeckmim added a comment to T12289: When a user is attending multiple overlapping events, computation of availability state may not be accurate.

rare

Maybe my calendar setup isn't common but I usually have multiple overlapping meetings each week - I know others at the same company manage their calendar in a similar fashion.

Feb 18 2017, 6:27 PM · Calendar, Bug Report

Feb 8 2017

cspeckmim added a comment to D17323: Add more phone numbers to "Shields Up" action.

Aha that makes sense. Thanks.

Feb 8 2017, 12:05 AM · Abuse
cspeckmim added a comment to D17323: Add more phone numbers to "Shields Up" action.

Out of curiosity, why are there "Context not available." indicators here? Am I not supposed to see R25 Secure?

Feb 8 2017, 12:01 AM · Abuse

Feb 7 2017

cspeckmim added a comment to T8937: Unclear exception when patching non-existant revision.

Is it possible this is related to the E argument mentioned in T12071, but for arc clients?

Feb 7 2017, 8:06 PM · Arcanist

Feb 4 2017

cspeckmim added a comment to Image Macro "micdrop".

is that chad

Feb 4 2017, 5:35 PM
cspeckmim added a comment to T12203: Remove Panel From Dashboard = UNRECOVERABLE FATAL ERROR.


It might be nice if the panel displayed ghosts instead of text tho

Feb 4 2017, 2:37 AM · Dashboards, Bug Report
cspeckmim added a comment to T12203: Remove Panel From Dashboard = UNRECOVERABLE FATAL ERROR.

Oh, the way I worded my original comment is confusing. I noticed the missing/unviewable panel on the homepage first then went to investigate and found error.

Feb 4 2017, 2:30 AM · Dashboards, Bug Report
cspeckmim added a comment to T12203: Remove Panel From Dashboard = UNRECOVERABLE FATAL ERROR.
Feb 4 2017, 2:26 AM · Dashboards, Bug Report
cspeckmim added a comment to T12203: Remove Panel From Dashboard = UNRECOVERABLE FATAL ERROR.

I just got this fatal error when trying to manage my dashboard on this install:
https://secure.phabricator.com/dashboard/manage/136/

Feb 4 2017, 2:23 AM · Dashboards, Bug Report

Feb 3 2017

cspeckmim added a comment to T12071: Require "E" be defined in variables_order so $_ENV is correctly populated.

My testing from D17296 indicates this would also resolve the original problem described in T9639

Feb 3 2017, 2:58 PM · Infrastructure
cspeckmim abandoned D17296: Pass mercurial PENDING environment into log command during prepushkey hook.

Thanks!

Feb 3 2017, 2:56 PM
cspeckmim added a comment to D17296: Pass mercurial PENDING environment into log command during prepushkey hook.

Adding E to variables_order seems to fix it!

  1. I reverted all files in the phabricator repository
  2. I stopped all phab services
  3. I modified /etc/php.ini to set variables_order = EGPCS
  4. I restarted all phab services
  5. On client cloned the mercurial repository
  6. On client I created a new commit and advanced the master bookmar
  7. On client I hg pus -r master
cspeck@speckimac ~/S/hgtest> hg pus -r master
pushing to ssh://repo-testing.code/source/hgtest/
searching for changes
remote: adding changesets
remote: adding manifests
remote: adding file changes
remote: added 1 changesets with 1 changes to 1 files
Feb 3 2017, 2:42 PM
cspeckmim added a comment to D17296: Pass mercurial PENDING environment into log command during prepushkey hook.

I will give that a try. My understanding of the problem/solution here is that when prepushkey hook is invoked by mercurial, it's within an environment that contains the changes of the transaction being requested, all of that being communicated via the environment variables. When this hook runs it shells out to execute some other mercurial commands to inspect the repository it loses that environment and is no longer operating within the environment of the transaction (resulting in the error about an unknown revision which is the one being pushed). I'm not really familiar with PHP, variables_order or $_ENV. Is the E thing for variables_order something that causes sub-processes to inherit the environment?

Feb 3 2017, 2:14 PM

Feb 2 2017

cspeckmim added a comment to T12139: Add a proper emoji picker to Remarkup bar.

Also I very probably meant "inline comment" and not "inline diff"

Feb 2 2017, 4:21 PM · Remarkup, PHUI
cspeckmim added a comment to T12139: Add a proper emoji picker to Remarkup bar.

I do not have persistent chat open.

Feb 2 2017, 4:19 PM · Remarkup, PHUI
cspeckmim added a comment to T12139: Add a proper emoji picker to Remarkup bar.

I came across a small bug with the "auto-suggest" feature. In differential I was writing an inline diff, and ended with :/ to show my face in plain ascii, and hit +enter immediately after to complete my inline diff. After typing :/ the auto-complete popped up and when I saved the draft with +enter it forever immortalized the auto-complete popup on screen with no seeming way to get rid of it (refreshing probably fixes~). I can reproduce on this install.

Feb 2 2017, 4:16 PM · Remarkup, PHUI
cspeckmim created D17296: Pass mercurial PENDING environment into log command during prepushkey hook.
Feb 2 2017, 7:51 AM
cspeckmim added a revision to T9639: Unable to push to hosted Mercurial repository over SSH with new upstream bookmark position: D17296: Pass mercurial PENDING environment into log command during prepushkey hook.
Feb 2 2017, 7:51 AM · Diffusion, Mercurial

Feb 1 2017

cspeckmim added a comment to T4124: Test the capabilities of `diff` better, especially for Solaris.

I've been seeing exceptions about "Can't parse an empty diff" in daemon logs, which I believe happens occasionally when merging commits and using a strategy to ignore changes from the incoming commit. This happens when merging a branch which might contain commits with similar/same changes which should not be kept due to already being resolved upstream. This results in any phabricator revisions/audits always showing the "Loading Content" when viewing in web.

Feb 1 2017, 9:27 PM · Arcanist
cspeckmim added a comment to T12180: Cropped text in Property List.

Almanac's is a bit tight too

Feb 1 2017, 4:49 AM · PHUI, Drydock, Bug Report
cspeckmim edited the description of T12180: Cropped text in Property List.
Feb 1 2017, 4:45 AM · PHUI, Drydock, Bug Report
cspeckmim edited the description of T12180: Cropped text in Property List.
Feb 1 2017, 4:45 AM · PHUI, Drydock, Bug Report
cspeckmim created T12180: Cropped text in Property List.
Feb 1 2017, 4:43 AM · PHUI, Drydock, Bug Report

Jan 30 2017

cspeckmim added a comment to T11708: Daemons erroneously saying "Diff does not exist".

I upgraded last Friday to 2604c5af55f654d36f8db2f080b96486c4572216, so far this exception has not popped up again. I will check again later in the week.

Jan 30 2017, 3:45 PM · Daemons, Differential, Bug Report

Jan 28 2017

cspeckmim added a comment to T9430: "Needs Audit" no longer excludes author commits.

Ah I guess it was a custom query - I checked someone else's dashboard who I know shows audits and theirs did not have the same issue as it was using a built-in one. Thanks for explaining, I can see how migrating a saved query would be difficult/impossible with no major benefits. I had never come across that type of error showing on page like that before and just wanted to check.

🐕

Jan 28 2017, 4:35 PM · Restricted Project, Restricted Project, Audit
cspeckmim added a comment to T9430: "Needs Audit" no longer excludes author commits.

I upgraded to 5efbf4d74aa72c5a7d7f161ea8c5aa33ac0e3189, after which when navigating to the list of audits on dashboard I get an error:

Exception: Query "need" is unknown to application search engine "PhabricatorCommitSearchEngine"!

The dashboard panel used the "Needs Audit" query (I believe was at least based on a default query) - is that expected for this? After modifying the dashboard to use a new panel using the new "Active Audits" query the dashboard correctly shows the results.

Jan 28 2017, 3:45 AM · Restricted Project, Restricted Project, Audit
cspeckmim added a comment to T11659: Differential comment panel haunter does not allow scrolling to bottom of differential revision when no ui.footer-items configured.

(Upgraded from 5efbf4d74aa72c5a7d7f161ea8c5aa33ac0e3189)

Jan 28 2017, 3:37 AM · Differential, Bug Report
cspeckmim added a comment to T11659: Differential comment panel haunter does not allow scrolling to bottom of differential revision when no ui.footer-items configured.

Confirmed this is fixed after upgrading to 864d3984ebdb42b9a2670103ca3607342e46b7dc

Jan 28 2017, 3:36 AM · Differential, Bug Report

Jan 27 2017

cspeckmim added a comment to T12136: What to do with "Application Counts" with MenuItem.

Yeah that makes sense. On mobile you don't get the dash but in that scenario you're probably wanting to look at most recently activity and not so much of "what do I have to do today".

Jan 27 2017, 8:42 PM · Profile Menu Item
cspeckmim added a comment to T12136: What to do with "Application Counts" with MenuItem.

Would it be the same effort to show that there are any vs. no items needing addressed?

Jan 27 2017, 8:35 PM · Profile Menu Item

Jan 25 2017

cspeckmim added a comment to T12153: 'Show Hidden Comments' bubble does not fit in margin next to line numbers >1000.

My first internship I was tasked with working on an application as sole developer. It started at 8.6k lines in a Main.cs file and at the end of the summer I left with it at about 15.5k. Luckily we didn't do code reviews.

Jan 25 2017, 11:24 PM · Differential, Bug Report
cspeckmim added a comment to T12139: Add a proper emoji picker to Remarkup bar.

hmm everything seems in order

Jan 25 2017, 4:34 AM · Remarkup, PHUI
cspeckmim added a comment to T12139: Add a proper emoji picker to Remarkup bar.

Click here for 🐱

Jan 25 2017, 4:30 AM · Remarkup, PHUI
cspeckmim added a comment to T12139: Add a proper emoji picker to Remarkup bar.

yup it works

Jan 25 2017, 4:24 AM · Remarkup, PHUI
cspeckmim added a comment to T12139: Add a proper emoji picker to Remarkup bar.

🐈

🐈
🐈

🐈

🐈

🐈

Jan 25 2017, 4:21 AM · Remarkup, PHUI
cspeckmim added a comment to T12139: Add a proper emoji picker to Remarkup bar.

cats are naturally dubious so i'm not surprised

Jan 25 2017, 4:05 AM · Remarkup, PHUI
cspeckmim added a comment to T12139: Add a proper emoji picker to Remarkup bar.

sort by awesome, descending

Jan 25 2017, 12:57 AM · Remarkup, PHUI
cspeckmim added a comment to T12139: Add a proper emoji picker to Remarkup bar.

Typing :cat will list the cat2 emoji before cat
🐱

Jan 25 2017, 12:55 AM · Remarkup, PHUI