Page MenuHomePhabricator
Feed Advanced Search

Nov 7 2014

chad added a comment to T6405: Support for Gmail Inbox actions.

Where did you run into that? I haven't seen it in the docs, offhand.

Nov 7 2014, 7:13 PM · Restricted Project, Phacility, Mail
avivey added a comment to T6405: Support for Gmail Inbox actions.

Note that there's a non-trivial per-install cost here (White-listing the application x domain with Google).

Nov 7 2014, 6:48 PM · Restricted Project, Phacility, Mail
chad claimed T6405: Support for Gmail Inbox actions.
Nov 7 2014, 6:46 PM · Restricted Project, Phacility, Mail
chad added a project to T6405: Support for Gmail Inbox actions: Mail.
Nov 7 2014, 6:44 PM · Restricted Project, Phacility, Mail
avivey updated subscribers of T6405: Support for Gmail Inbox actions.
Nov 7 2014, 6:43 PM · Restricted Project, Phacility, Mail

Nov 3 2014

chad lowered the priority of T5382: Notifications missing from "Slowvote" votes from Normal to Wishlist.
Nov 3 2014, 2:52 AM · Slowvote

Oct 29 2014

chad removed a project from T6405: Support for Gmail Inbox actions: Audit.
Oct 29 2014, 4:32 AM · Restricted Project, Phacility, Mail

Oct 28 2014

epriestley triaged T6405: Support for Gmail Inbox actions as Wishlist priority.
Oct 28 2014, 11:21 AM · Restricted Project, Phacility, Mail
aik099 created T6405: Support for Gmail Inbox actions.
Oct 28 2014, 8:29 AM · Restricted Project, Phacility, Mail

Oct 21 2014

aik099 added a comment to T6359: Show only unread notifications in notification baloon.

Also I believe, that in Notifications app the "Unread Notifications" query should be default, not "All Notifications" as it is now.

Oct 21 2014, 2:36 PM · Notifications
chad added a comment to T6359: Show only unread notifications in notification baloon.

You may modify your instance of Phabricator via forking or patching to suite your individual needs.

Oct 21 2014, 2:31 PM · Notifications
aik099 added a comment to T6359: Show only unread notifications in notification baloon.

Isn't having "Mark as Read" link on list of already read notifications controversial? Usually when people click on something, then something should happen/change, but it doesn't in this case.

Oct 21 2014, 2:21 PM · Notifications
chad closed T6359: Show only unread notifications in notification baloon as Wontfix.

We're not interested in pursuing this in the upstream as the information is useful regardless of if it's been read or not. It's also consistent with how other notification systems work (Facebook, Twitter, etc). Thanks for the feedback.

Oct 21 2014, 2:16 PM · Notifications
aik099 created T6359: Show only unread notifications in notification baloon.
Oct 21 2014, 9:15 AM · Notifications

Oct 15 2014

chad added a comment to T6309: Build Notification "Switchboard" Preference Panel.

Nuance is our future product intended to let installs interact with common folk (for issues) anywhere on the web.

Oct 15 2014, 4:00 PM · Notifications
epriestley added a comment to T6309: Build Notification "Switchboard" Preference Panel.

Is this documented somewhere?

Oct 15 2014, 3:57 PM · Notifications
nemobis added a comment to T6309: Build Notification "Switchboard" Preference Panel.

This software is also not targeted at consumers or the public at large. It is targeted at professionals in software fields and adjacent fields.

Oct 15 2014, 3:26 PM · Notifications
epriestley added a comment to T6309: Build Notification "Switchboard" Preference Panel.

This software is also not targeted at consumers or the public at large. It is targeted at professionals in software fields and adjacent fields. Product and prioritization decisions we make in the upstream reflect that.

Oct 15 2014, 2:42 PM · Notifications
chad added a comment to T6309: Build Notification "Switchboard" Preference Panel.

Then an interface is needed to set the site defaults for those.

Oct 15 2014, 2:31 PM · Notifications
nemobis added a comment to T6309: Build Notification "Switchboard" Preference Panel.

If I look at https://secure.phabricator.com/settings/panel/emailpreferences/ , this seems rather easy conceptually: there is already one row (and selector) for each possible event, they only need to be "exploded" by the possible relations one can have to the event: Actions x {I made it; or not} x {I'm author of the task/whatever; I'm subscribed to it; I'm assigned to the task; I'm backer of the fund proposal; ... }. You already have the first two combinations, just need the latter.

Oct 15 2014, 11:47 AM · Notifications

Oct 14 2014

epriestley added a comment to T6309: Build Notification "Switchboard" Preference Panel.

I don't think a switchboard of checkboxes can accommodate requests like T6296.

Oct 14 2014, 9:11 PM · Notifications
chad updated subscribers of T6309: Build Notification "Switchboard" Preference Panel.
Oct 14 2014, 9:02 PM · Notifications
chad created T6309: Build Notification "Switchboard" Preference Panel.
Oct 14 2014, 9:02 PM · Notifications

Sep 14 2014

aklapper added a comment to T5231: Allow Broadcasting on all channels.

@chase.mp told me on Sep 07 "there is no email broadcast, I had to write one" but I don't know any details either.

Sep 14 2014, 8:43 PM · Notifications

Sep 9 2014

qgil added a project to T5231: Allow Broadcasting on all channels: Notifications.
Sep 9 2014, 8:54 AM · Notifications

Sep 5 2014

epriestley added a comment to T5982: Notifications: "Mark all Read" not clearing unread notifications list.

Cool, thanks!

Sep 5 2014, 2:42 AM · Notifications
• mholden added a comment to T5982: Notifications: "Mark all Read" not clearing unread notifications list.

Sorry I never got around to testing that temporary fix.

Sep 5 2014, 2:40 AM · Notifications

Sep 4 2014

epriestley closed T5982: Notifications: "Mark all Read" not clearing unread notifications list as Resolved.

Closed by commit rP4d3cc7b28dd1.

Sep 4 2014, 7:48 PM · Notifications
epriestley added a revision to T5982: Notifications: "Mark all Read" not clearing unread notifications list: Restricted Differential Revision.
Sep 4 2014, 6:02 PM · Notifications

Sep 2 2014

epriestley added a comment to T5982: Notifications: "Mark all Read" not clearing unread notifications list.

2147483647 is the value on 32-bit systems, but %d may not work properly with 64-bit integers on a 32-bit system. I'm guessing the actual query we're running is chronologicalKey < 12 (for some incorrect, 32-bit-coerced value of 12) instead of chronlogicalKey < 6052774592759489916.

Sep 2 2014, 7:20 PM · Notifications
• mholden added a comment to T5982: Notifications: "Mark all Read" not clearing unread notifications list.

Sure thing; I'll need to wait until tonight since we've got people using our install throughout the day, and I still haven't set up any sort of 'test' environment.

Sep 2 2014, 7:16 PM · Notifications
btrahan added a comment to T5982: Notifications: "Mark all Read" not clearing unread notifications list.

Wow, i didn't realize PHP could have super different values for PHP_INT_MAX beyond like 32 bit vs 64 bit OS and stuff like that.

Sep 2 2014, 7:15 PM · Notifications
epriestley added a comment to T5982: Notifications: "Mark all Read" not clearing unread notifications list.

Aha! Can you try this patch and see if it works?

Sep 2 2014, 7:07 PM · Notifications
• mholden added a comment to T5982: Notifications: "Mark all Read" not clearing unread notifications list.

2147483647

Sep 2 2014, 7:02 PM · Notifications
epriestley added a comment to T5982: Notifications: "Mark all Read" not clearing unread notifications list.

@mholden, can you show us the output of this command on your system?

Sep 2 2014, 6:04 PM · Notifications
btrahan added a comment to T5982: Notifications: "Mark all Read" not clearing unread notifications list.

Just to clarify this issue is on a single instance only. I think @mholden has provided evidence that somehow the sql query which should work isn't. I am not sure what ghosts could be living in this specific instance, and of course perhaps this is just the first instance of this problem we've seen reported...

Sep 2 2014, 5:30 PM · Notifications
chad triaged T5982: Notifications: "Mark all Read" not clearing unread notifications list as Normal priority.
Sep 2 2014, 5:28 PM · Notifications
btrahan placed T5982: Notifications: "Mark all Read" not clearing unread notifications list up for grabs.

@epriestley - I am wildly stumped on this one. Code should work; does not.

Sep 2 2014, 5:17 PM · Notifications

Aug 30 2014

• mholden added a comment to T5982: Notifications: "Mark all Read" not clearing unread notifications list.

Sorry I took a day to get back.

Aug 30 2014, 6:18 AM · Notifications

Aug 28 2014

btrahan added a comment to T5982: Notifications: "Mark all Read" not clearing unread notifications list.

To cut to the chase a bit, can you talk more about the "magic" that you might have done to your instance? I don't think we should bother debugging stuff if you're confident its something you did to your instance.

Aug 28 2014, 8:04 PM · Notifications
• mholden added a comment to T5982: Notifications: "Mark all Read" not clearing unread notifications list.

If it would help, I can paste the results of any SQL I can run that might help you see what's going on..?

Aug 28 2014, 7:35 PM · Notifications
btrahan added a comment to T5982: Notifications: "Mark all Read" not clearing unread notifications list.

Hmmm, I can't seem to repro this. Would the "magic" involve making things with weird timestamps?

Aug 28 2014, 7:26 PM · Notifications
btrahan claimed T5982: Notifications: "Mark all Read" not clearing unread notifications list.

Thanks for the report. I am 99% sure this is some fallout from fixing T5975.

Aug 28 2014, 4:34 PM · Notifications
• mholden added a project to T5982: Notifications: "Mark all Read" not clearing unread notifications list: Notifications.

Blindly associating this with the "Notifications" project because that sounds like it would make sense.

Aug 28 2014, 12:20 AM · Notifications

Aug 27 2014

btrahan closed T5975: Next button in notifications page doesn't work as Resolved.

Closed by commit rP85b767bbdc65.

Aug 27 2014, 7:20 PM · Notifications
btrahan claimed T5975: Next button in notifications page doesn't work.
Aug 27 2014, 7:14 PM · Notifications
btrahan added a revision to T5975: Next button in notifications page doesn't work: Restricted Differential Revision.
Aug 27 2014, 7:14 PM · Notifications
chad triaged T5975: Next button in notifications page doesn't work as Normal priority.
Aug 27 2014, 5:12 PM · Notifications

Aug 16 2014

epriestley added a revision to T5891: Add button to notification panel to concatenate notifications on the same objects: Restricted Differential Revision.
Aug 16 2014, 3:04 AM · Notifications
chad triaged T5891: Add button to notification panel to concatenate notifications on the same objects as Low priority.
Aug 16 2014, 2:02 AM · Notifications
chad added a comment to T5891: Add button to notification panel to concatenate notifications on the same objects.

I fear this wouldn't make a busy install much less busy. I like the idea of more of a 'synopsis' type feed, as beyond just people on vacation, could be beneficial to people outside of engineering. Some sort of daily summary with links to queries (5 tasks opened today, 12 tasks closed, 5 diffs committed, yada yada yada).

Aug 16 2014, 1:29 AM · Notifications
epriestley added a comment to T5891: Add button to notification panel to concatenate notifications on the same objects.

(We could also restore aggregation but use it only in feed, or only in notifications. All of the options here feel about equal to me for how I use the product, at least.)

Aug 16 2014, 1:17 AM · Notifications
epriestley updated subscribers of T5891: Add button to notification panel to concatenate notifications on the same objects.

Tangentially related: we have some story aggregation code in the codebase which we used to use a long time ago. It aggregates notifications and feed stories (specifically: multiple updates to the same object by one or more editors in roughly the same period of time) into stories like this:

Aug 16 2014, 1:15 AM · Notifications
epriestley added a project to T5891: Add button to notification panel to concatenate notifications on the same objects: Notifications.
Aug 16 2014, 12:57 AM · Notifications

Aug 15 2014

chad closed T5575: Connection info in Notifications header needs rethinking as Resolved.

Closed by commit rPdc69c4e58c2a.

Aug 15 2014, 12:19 AM · Design, Notifications

Aug 14 2014

chad added a revision to T5575: Connection info in Notifications header needs rethinking: Restricted Differential Revision.
Aug 14 2014, 8:29 PM · Design, Notifications
webframp edited P1233 aphlict crash - phabricator version 1e1eb6921b1df7ad4f53a388549d3714853d2730.
Aug 14 2014, 12:52 PM · Notifications

Aug 13 2014

chad closed T5537: Notification panel does not resize itself to browser window height as Resolved.

Closed by commit rPc740d554f667.

Aug 13 2014, 10:37 PM · Notifications
chad added a revision to T5537: Notification panel does not resize itself to browser window height: Restricted Differential Revision.
Aug 13 2014, 10:35 PM · Notifications

Aug 12 2014

epriestley added a comment to T5769: Add a setting to not trigger notifications when moving task across workboard columns.

Specifically: Settings > Email Preferences > Maniphest Tasks > "A task is moved between columns on a workboard." > Ignore

Aug 12 2014, 9:16 PM · Notifications
epriestley closed T5769: Add a setting to not trigger notifications when moving task across workboard columns as Resolved.

Closed by commit rP8ef1ea63ddb9.

Aug 12 2014, 9:07 PM · Notifications
epriestley added a revision to T5769: Add a setting to not trigger notifications when moving task across workboard columns: Restricted Differential Revision.
Aug 12 2014, 8:52 PM · Notifications