Page MenuHomePhabricator
Feed All Stories

Dec 16 2015

epriestley added a comment to T9999: EditEngine "Space" field is configurable as visible/locked/hidden, but should always have the same value as "View Policy".

That is, I changed the configuration to fix things on this install; the new task details describe fixing the root issue.

Dec 16 2015, 3:29 PM · Bug Report
epriestley added a comment to T9999: EditEngine "Space" field is configurable as visible/locked/hidden, but should always have the same value as "View Policy".

This was a configuration issue which permitted configuring a currently-invalid mixture of "Space" and "View Policy" visibility.

Dec 16 2015, 3:29 PM · Bug Report
epriestley renamed T9999: EditEngine "Space" field is configurable as visible/locked/hidden, but should always have the same value as "View Policy" from "You must choose a space for this object" while using Maniphest form to EditEngine "Space" field is configurable as visible/locked/hidden, but should always have the same value as "View Policy".
Dec 16 2015, 3:28 PM · Bug Report
epriestley closed T10002: Test for T9999 as Spite.
Dec 16 2015, 3:26 PM · Feature Request
epriestley closed T10001: Test for T9999 as Spite.
Dec 16 2015, 3:26 PM · Bug Report
epriestley created T10002: Test for T9999.
Dec 16 2015, 3:26 PM · Feature Request
epriestley created T10001: Test for T9999.
Dec 16 2015, 3:26 PM · Bug Report
epriestley added a revision to T9964: Fill in Conduit support infrastructure for ApplicationEditor changes: Unknown Object (Differential Revision).
Dec 16 2015, 3:17 PM · Maniphest, Phame, Owners, Arcanist, Conduit
chad awarded T10000: Consider restoring Clowncopterize a Like token.
Dec 16 2015, 3:10 PM · Differential, Feature Request
cburroughs awarded T10000: Consider restoring Clowncopterize a Like token.
Dec 16 2015, 3:08 PM · Differential, Feature Request
chad created T10000: Consider restoring Clowncopterize.
Dec 16 2015, 3:03 PM · Differential, Feature Request
chad added a comment to T8872: phabricator admin Login Failure.

@Suhasini_thakur The message tells you how to fix it. Did you attempt to follow the instructions it provided?

Dec 16 2015, 3:00 PM · Auth
agenticarus updated Q250: Can I treat some users as second class citizens? from to Can I treat some users as second class citizens?.
Dec 16 2015, 2:23 PM · Differential
epriestley added a revision to T9993: arc land incorrectly reports I used `--onto`: Unknown Object (Differential Revision).
Dec 16 2015, 2:02 PM · Arcanist, Bug Report
epriestley added a revision to T9964: Fill in Conduit support infrastructure for ApplicationEditor changes: Unknown Object (Differential Revision).
Dec 16 2015, 1:52 PM · Maniphest, Phame, Owners, Arcanist, Conduit
Suhasini_thakur added a comment to T8872: phabricator admin Login Failure.

Phabricator is installed properly.But after installation it shows some error in mysql.

Dec 16 2015, 11:48 AM · Auth
nnnn20430 added a comment to T8180: Notifications UI stuck in "Setting Up Client" after configuring websockets & aphlic.

@ablekh i just added a "fastcgi_param HTTPS on;" in nginx config for phabricator

Dec 16 2015, 10:40 AM · Notifications
ablekh awarded T9293: WebSockets access problems with reverse SSL proxy a Love token.
Dec 16 2015, 9:57 AM · Restricted Project, Aphlict
ablekh added a comment to T8180: Notifications UI stuck in "Setting Up Client" after configuring websockets & aphlic.

@nnnn20430: Thank you for the reference. Did you apply the patch, included in T9293: WebSockets access problems with reverse SSL proxy ?

Dec 16 2015, 9:55 AM · Notifications
Funnnny added a comment to T9905: Upgrading: ApplicationEditor changes in Maniphest.

@ox: stable is very close to the edge. I think if you want production use, you should just use stable

Dec 16 2015, 8:14 AM · Maniphest, Installing & Upgrading
nnnn20430 added a comment to T8180: Notifications UI stuck in "Setting Up Client" after configuring websockets & aphlic.

ok figured it out the problem for me was because of this bug T9293

Dec 16 2015, 8:05 AM · Notifications
chad added a comment to T8872: phabricator admin Login Failure.

http://www.tecmint.com/install-phabricator-in-linux/ should not be followed. It is an out of date guide. We only support installing from our guide.

Dec 16 2015, 7:04 AM · Auth
chad added a comment to T8872: phabricator admin Login Failure.

I created user from browser but it was not working then i created user from command line with another user.

Dec 16 2015, 7:02 AM · Auth
Suhasini_thakur added a comment to T8872: phabricator admin Login Failure.

I have installed phabricator from https://secure.phabricator.com/book/phabricator/article/installation_guide/ and www.tecmint.com/install-phabricator-in-linux/
I did not create any user from terminal before visiting it in the browser.I created user from browser but it was not working then i created user from command line with another user.
Please let me know about this.

Dec 16 2015, 6:56 AM · Auth
chad added a comment to T8872: phabricator admin Login Failure.

How did you install Phabricator? Did you create a user from the command line before visiting it in the browser?

Dec 16 2015, 6:20 AM · Auth
Suhasini_thakur added a comment to T8872: phabricator admin Login Failure.

Hello,
I have installed Phabricator on ubuntu 14.04.But when I enter phabricator.domain.com,it shows an authentication error "Authentication Failure
This Phabricator install is not configured with any enabled authentication providers which can be used to log in. If you have accidentally locked yourself out by disabling all providers, you can use phabricator/bin/auth recover <username> to recover access to an administrative account."

Dec 16 2015, 5:57 AM · Auth
hach-que added a comment to T9999: EditEngine "Space" field is configurable as visible/locked/hidden, but should always have the same value as "View Policy".

@epriestley I sure hope there's a badge for whoever creates T10000...

Dec 16 2015, 5:16 AM · Bug Report
hach-que created T9999: EditEngine "Space" field is configurable as visible/locked/hidden, but should always have the same value as "View Policy".
Dec 16 2015, 5:15 AM · Bug Report
hach-que created T9998: Surface AWS SES error messages better.
Dec 16 2015, 5:15 AM · Feature Request
cobraBJ added a comment to T6361: Unhandled Exception When Trying to View Diff in Phriction.

I got the same problem :

Dec 16 2015, 3:08 AM

Dec 15 2015

ox added a comment to T9905: Upgrading: ApplicationEditor changes in Maniphest.

Is there an expected date when this change will be safe for production use? That red warning bar at the top of the summary is very intimidating, though I want to keep our instance responsibly close to the edge (should I just use stable?).

Dec 15 2015, 11:25 PM · Maniphest, Installing & Upgrading
epriestley added a comment to T9997: Allow login to be disabled for an authentication provider.

(The first change attached here was the instance sync change.)

Dec 15 2015, 11:09 PM · Auth, Phacility
epriestley added a comment to T9997: Allow login to be disabled for an authentication provider.

This was easier than I thought and now in HEAD. Should go out Saturday unless churn related to T9905 makes me uneasy about promoting to stable -- I'm currently feeling good about promoting, but still have three full days to mangle master.

Dec 15 2015, 11:09 PM · Auth, Phacility
epriestley closed T9752: Arbitrary values allowed for repo PHID in when adding a path in owners as Resolved by committing Restricted Diffusion Commit.
Dec 15 2015, 11:04 PM · Owners
turadg awarded T9997: Allow login to be disabled for an authentication provider a Haypence token.
Dec 15 2015, 11:03 PM · Auth, Phacility
epriestley closed T6205: Replace "Associate Projects" action in task editing form with "Edit Projects" as Resolved by committing Restricted Diffusion Commit.
Dec 15 2015, 11:03 PM · Projects, Transactions, Maniphest
epriestley closed T9997: Allow login to be disabled for an authentication provider as Resolved by committing Restricted Diffusion Commit.
Dec 15 2015, 11:03 PM · Auth, Phacility
epriestley closed T9997: Allow login to be disabled for an authentication provider, a subtask of T7441: Support OAuth providers in the cluster, as Resolved.
Dec 15 2015, 11:03 PM · Auth, Phacility
epriestley added a revision to T9997: Allow login to be disabled for an authentication provider: Unknown Object (Differential Revision).
Dec 15 2015, 10:52 PM · Auth, Phacility
epriestley added a revision to T9997: Allow login to be disabled for an authentication provider: Unknown Object (Differential Revision).
Dec 15 2015, 10:34 PM · Auth, Phacility
nnnn20430 added a comment to T8180: Notifications UI stuck in "Setting Up Client" after configuring websockets & aphlic.

i have same problem, the notification server is working and it logs notifications like on new message in Conpherence app, but nothing get's updated in notifications ui and it also just says "Setting Up Client".

Dec 15 2015, 10:31 PM · Notifications
Taskle added a comment to T9995: Blog posts that include a "/" character show a 404 error.

Verified! You guys rock!

Dec 15 2015, 10:25 PM · Phame, Bug Report
epriestley added a comment to T9995: Blog posts that include a "/" character show a 404 error.

This should be fixed in HEAD. Let us know if you're still seeing issues or we missed anything.

Dec 15 2015, 10:23 PM · Phame, Bug Report
epriestley closed T9995: Blog posts that include a "/" character show a 404 error as Resolved by committing Restricted Diffusion Commit.
Dec 15 2015, 10:19 PM · Phame, Bug Report
epriestley created T9997: Allow login to be disabled for an authentication provider.
Dec 15 2015, 9:51 PM · Auth, Phacility
epriestley reopened T9997: Allow login to be disabled for an authentication provider, a subtask of T7441: Support OAuth providers in the cluster, as Open.
Dec 15 2015, 9:51 PM · Auth, Phacility
Taskle added a comment to T9995: Blog posts that include a "/" character show a 404 error.

Even if it's not an everyone has a blog case, again, I can't imagine it's a common use case to regularly be switching blogs to post on. Again, this is just a convenience function (a "New Post" button that creates a new post for the last-posted-to-blog). Not required, just a nice to have.

Dec 15 2015, 9:47 PM · Phame, Bug Report
aleb added a comment to T9987: Diffs in a task should be sorted by the dependency tree.

We use git-phab, a tool which creates a task per branch and keeps the commits in the branch and the diffs of the task in sync. https://phabricator.freedesktop.org/diffusion/GITPHAB/ What this means is that it allows us to change the order of the diffs more easily, so in our case it will happen more often!

Dec 15 2015, 9:45 PM · Maniphest, Feature Request
chad added a comment to T9995: Blog posts that include a "/" character show a 404 error.

Ah ok, I don't think we've considered the "everyone has a blog" case. It might get hairy.

Dec 15 2015, 9:44 PM · Phame, Bug Report
Taskle added a comment to T9995: Blog posts that include a "/" character show a 404 error.

Oh cool - that's a useful shortcut. But imagine that the primary use case is this sort of note taking where everyone has their personal blog, I anticipate that 95% of the time, people will be writing blog posts on the same blog they last wrote on. For the 15 people on our team, 100% of us only write on one blog which is our personal blog.

Dec 15 2015, 9:42 PM · Phame, Bug Report
epriestley added a revision to T9995: Blog posts that include a "/" character show a 404 error: Unknown Object (Differential Revision).
Dec 15 2015, 9:41 PM · Phame, Bug Report
chad added a comment to T9995: Blog posts that include a "/" character show a 404 error.

The + is a link to create new posts, may not be 100% obvious.

pasted_file (353×320 px, 35 KB)

Dec 15 2015, 9:39 PM · Phame, Bug Report
Taskle added a comment to T9995: Blog posts that include a "/" character show a 404 error.

The only feedback I have is that since the "New Blog" button was added top Right (instead of, for example, "New Post" which would default to, say, the last blog that you posted to), I inadvertently created a new Blog when I meant to write a post.

Dec 15 2015, 9:37 PM · Phame, Bug Report
chad added a comment to T9995: Blog posts that include a "/" character show a 404 error.

Sounds like you're using it as we do, though.

Dec 15 2015, 9:35 PM · Phame, Bug Report
chad added a comment to T9995: Blog posts that include a "/" character show a 404 error.

Well we're about to unbeta Phame if you have any use cases we're not thinking of.

Dec 15 2015, 9:35 PM · Phame, Bug Report
Taskle added a comment to T9995: Blog posts that include a "/" character show a 404 error.

As an aside, if you guys ever want insight from heavy Phab users, let me and Nico know! We're so deeply integrated now...we even automate the creation of Phab tasks via Salesforce to manage our entire Client Services team's work flow to onboard new clients. :)

Dec 15 2015, 9:34 PM · Phame, Bug Report
Taskle added a comment to T9995: Blog posts that include a "/" character show a 404 error.

Awesome :) yes, I don't see any of us caring about the slugs. In fact I remember it being a bit of a nuisance to do this because we'd have 15 people all with the slug of, for example, 12/10/2016, so we'd all have to make them unique. :)

Dec 15 2015, 9:33 PM · Phame, Bug Report
chad added a comment to T9995: Blog posts that include a "/" character show a 404 error.

ah, true

Dec 15 2015, 9:30 PM · Phame, Bug Report
epriestley added a comment to T9995: Blog posts that include a "/" character show a 404 error.

Since slugs no longer need to be unique, I'm just going to simplify this whole process and generate them automatically from titles.

Dec 15 2015, 9:30 PM · Phame, Bug Report
techtonik added a comment to T7073: The meaning of the login icon is not clear.

When I first wanted to report a bug to a project that uses Phabricator, it took me several minutes (!) to figure out how to login and I'm used to all kinds of bug-trackers (mantis, jira, trac, ...).

Dec 15 2015, 9:30 PM
techtonik awarded T7073: The meaning of the login icon is not clear a Haypence token.
Dec 15 2015, 9:28 PM
epriestley added a revision to T9964: Fill in Conduit support infrastructure for ApplicationEditor changes: Unknown Object (Differential Revision).
Dec 15 2015, 9:26 PM · Maniphest, Phame, Owners, Arcanist, Conduit
Taskle added a comment to T9995: Blog posts that include a "/" character show a 404 error.

We don't have external references to Phame, only internal references within Phame itself. So no need to have the old urls work in that regard. It's just that clicking the links in the phame UI itself is an important piece we'll need to work.

Dec 15 2015, 9:22 PM · Phame, Bug Report
nickz updated subscribers of T9994: Merge conflicts causes resources not being able to be released.
Dec 15 2015, 9:20 PM · Drydock, Differential, Bug Report
epriestley added a comment to T9995: Blog posts that include a "/" character show a 404 error.

Also, is the fact that the URL scheme changed an issue (i.e., do you have a lot of links to the old URLs in external systems)? We weren't aware anyone was really using Phame seriously, but I can write a little extension you could use to keep the old redirects working for a while if that's also causing you pain.

Dec 15 2015, 9:20 PM · Phame, Bug Report
Taskle added a comment to T9995: Blog posts that include a "/" character show a 404 error.

(the reason this issue is exacerbated for us is because the titles of all our posts are dates, which many folks use /'s for, so we have a ton of posts with this issue)

Dec 15 2015, 9:16 PM · Phame, Bug Report
Taskle added a comment to T9995: Blog posts that include a "/" character show a 404 error.

Yeah it's a critical part of our workflow :) back at Google there's this concept of "weekly snippets" that the whole company used to use and our engineering team here at Doctor.com models after that. It's really useful to get visibility into work. :)

Dec 15 2015, 9:16 PM · Phame, Bug Report
epriestley added a comment to T9995: Blog posts that include a "/" character show a 404 error.

We could also just bring that patch upstream, but I think allowing "/" in slugs is probably a decision we'd regret in the long term.

Dec 15 2015, 9:15 PM · Phame, Bug Report
epriestley added a comment to T9995: Blog posts that include a "/" character show a 404 error.

As a workaround, you can apply this patch locally, which should make things work for now:

Dec 15 2015, 9:15 PM · Phame, Bug Report
chad added a comment to T9995: Blog posts that include a "/" character show a 404 error.

I am stoked someone uses Phame though!

Dec 15 2015, 9:13 PM · Phame, Bug Report
chad added a comment to T9995: Blog posts that include a "/" character show a 404 error.

(maybe we can hack the route?)

Dec 15 2015, 9:12 PM · Phame, Bug Report
chad updated subscribers of T9995: Blog posts that include a "/" character show a 404 error.

I don't know how involved a migration script would be, @epriestley?

Dec 15 2015, 9:11 PM · Phame, Bug Report
chad added a comment to T9995: Blog posts that include a "/" character show a 404 error.

You can use the URL /phame/post/edit/775/

Dec 15 2015, 9:10 PM · Phame, Bug Report
Taskle added a comment to T9995: Blog posts that include a "/" character show a 404 error.

@chad Hi Chad, so how can we remove it if we can't edit the post? Is there a workaround to edit the post? Or some manual database change we can make? We have literally hundreds of blog posts as we use this feature internally to track weekly work (hence the urgency as now our entire company has lost visibility into each other's work), so it would be a nightmare to do manually...

Dec 15 2015, 9:10 PM · Phame, Bug Report
chad added a comment to T9987: Diffs in a task should be sorted by the dependency tree.

T5815 and T9372 are all roughly along these lines to providing the experience you want. It'd also be good to know what's missing from Differential's QueryEngine that you're manually adding things to tasks.

Dec 15 2015, 9:01 PM · Maniphest, Feature Request
chad added a comment to T9987: Diffs in a task should be sorted by the dependency tree.

Based on https://phabricator.freedesktop.org/T3491 the root problem to me looks like "it should be easier to review large batchs of diffs". Right now you're using Maniphest as a stop-gap solution that should be correctly resolved Differential's Custom Query

Dec 15 2015, 8:57 PM · Maniphest, Feature Request
epriestley closed T9996: Comment box should be above the differential as Wontfix.

This is an intentional decision.

Dec 15 2015, 8:43 PM · Feature Request
chad added a comment to T9995: Blog posts that include a "/" character show a 404 error.

You'll need to edit your title phame_title to fix the 404 for now (I don't think it's worth providing a retroactive fix). We should have always been removing / from Phame Title as it would have always 404'd with Live View.

Dec 15 2015, 8:37 PM · Phame, Bug Report
kyle.ross created T9996: Comment box should be above the differential.
Dec 15 2015, 8:37 PM · Feature Request
dereckson added a comment to T8768: Under Mercurial, `arc patch` mishandles files with spaces in them .

@michaeljs1990. You seem to have another issue.

Dec 15 2015, 8:24 PM · Git, Mercurial, Arcanist
nickz added a comment to T9994: Merge conflicts causes resources not being able to be released.

It looks like it counts the total working copies. I thought the limit is for every single repo: when I set the the limit to be 5, I expect that every single repo can have 5 copies at most. This is not scalable: when more and more repos get created, we have to adjust the limit.

Dec 15 2015, 7:56 PM · Drydock, Differential, Bug Report
nickz added a comment to T9994: Merge conflicts causes resources not being able to be released.

Also why can't a new lease get another resource when previous lease holds a resource? I set the limit of the working copy to be 5, so there should be enough resources available to be used.

Dec 15 2015, 7:35 PM · Drydock, Differential, Bug Report
nicolast updated subscribers of T9995: Blog posts that include a "/" character show a 404 error.
Dec 15 2015, 7:33 PM · Phame, Bug Report
nicolast created T9995: Blog posts that include a "/" character show a 404 error.
Dec 15 2015, 7:32 PM · Phame, Bug Report
nickz created T9994: Merge conflicts causes resources not being able to be released.
Dec 15 2015, 7:12 PM · Drydock, Differential, Bug Report
foven added a comment to Q180: Why does SVN Commit fail with blocked by pre-commit?.

@silverskysoft that's unfortunate, but thanks for letting me know.

Dec 15 2015, 7:09 PM · Subversion
aleb added a comment to T9987: Diffs in a task should be sorted by the dependency tree.

When the reviewer looks at the task to review the diffs, he needs to start somewhere and end somewhere. This is where the diffs list help: the reviewer will start at the top and finish at the bottom. Currently the diffs are ordered by creation time. Normally this works ok because the newer diffs depend on the older diffs. But if a newer diff is made a dependency of an older diff, the order of the list is not relevant.

Dec 15 2015, 6:52 PM · Maniphest, Feature Request
csilvers added a comment to T9993: arc land incorrectly reports I used `--onto`.

w00t w00t, I got something out of this bug report after all!

Dec 15 2015, 6:38 PM · Arcanist, Bug Report
epriestley edited Description on Community.
Dec 15 2015, 6:37 PM
epriestley added a comment to T9993: arc land incorrectly reports I used `--onto`.

I inducted you into Community so you should have sweeping editorial powers once again.

Dec 15 2015, 6:37 PM · Arcanist, Bug Report
csilvers closed T9993: arc land incorrectly reports I used `--onto` as Invalid.
Dec 15 2015, 6:37 PM · Arcanist, Bug Report
epriestley added a member for Community: csilvers.
Dec 15 2015, 6:36 PM
csilvers added a comment to T9993: arc land incorrectly reports I used `--onto`.

I don't have a shell alias, but I do have a wrapper script around arc. Doh! I totally forgot about that. Sorry for the false alarm.

Dec 15 2015, 6:36 PM · Arcanist, Bug Report
epriestley added a comment to T9993: arc land incorrectly reports I used `--onto`.

I can't reproduce this:

Dec 15 2015, 6:25 PM · Arcanist, Bug Report
csilvers created T9993: arc land incorrectly reports I used `--onto`.
Dec 15 2015, 6:19 PM · Arcanist, Bug Report
michaeljs1990 added a comment to T8768: Under Mercurial, `arc patch` mishandles files with spaces in them .

I am curious about this as well or if it has been addressed some other place. It's pretty easy to manually resolve the issue most of the time however I am not sure what the proper action would be after doing this? Does phabricator expect you to arc diff after you manually resolve it?

Dec 15 2015, 5:50 PM · Git, Mercurial, Arcanist
chad added a comment to T9992: Build a general-purpose icon selection control.

awwyiss

Dec 15 2015, 5:41 PM
epriestley created T9992: Build a general-purpose icon selection control.
Dec 15 2015, 5:36 PM
epriestley closed T9991: testibus as Spite.
Dec 15 2015, 5:29 PM · Feature Request
epriestley closed T9991: testibus, a subtask of T9990: testibus, as Spite.
Dec 15 2015, 5:29 PM · Feature Request