Trying to make Phabricator better every day. ʕ•ᴥ•ʔ
- User Since
- Oct 27 2011, 3:22 AM (273 w, 2 d)
In pretty much all cases, I’d rather put time into fixing dashboards or
building Nuance, etc, if the count behind something is important.
- remove php code
seems like we could easily integrate this into MenuItem. I didn't know about loadStatus
I can vanish them now.
Oh I guess we can pre set the names in code as bulletins here.
The main reason I hadn't done the app name thing yet was because I planned to look into alternate names when I redo Home. I don't know if this is in addition to maybe having a "menu item name" per application or just sidesteps us renaming them. I don't know if that option you suggested was still on the table. Maybe we talked about this in person, I can't recall.
Yeah I meant to clean that up too
@epriestley my assumption here is:
@Ondrej I was mostly trying to understand if you were seeing a regression, a mis-understanding of how Audit currently works, or missing some setting we should be trying against a default install. There were a number of variables still open based on your explanation to me at least, sounds like @epriestley has plans forward here though already.
Well there go my morning plans.
if a concern is raised; no one except this auditor can close the audit.
I did for now, will see what FontAwesome 5 brings, but the was never something I really liked for Phame anyways.
Beyond that I don't know how you're using Calendar or what types of events you have.
I'd update Phabricator first, then see if it clears.
We will also need reproduction steps to accept this as a valid bug report. See Contributing Bug Reports.
You're at least 6 weeks out of date. Can you update Phabricator and verify the bug at HEAD before filing a bug, please?
Thu, Jan 19
I’m guessing that’s not something I could do?
(I've grown to like the icon-less menus though)
We'd probably be fine picking 16 per form... Not sure people customize forms outside of Maniphest though.
How should we have caught this? Non-admin testing?
And before you file it, no, cat facts are not supported.
This is complete for now, there may be more minor updates, but all top level goals have shipped.
This is in a reasonable state, will follow up with a new task on errata.
Wed, Jan 18
This is a bug that has been resolved in the upstream, please update to HEAD.
Still running into this a little.