New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add android wear support #619

Closed
vinilox opened this Issue Apr 21, 2015 · 24 comments

Comments

Projects
None yet
10 participants
@vinilox

vinilox commented Apr 21, 2015

Gmail has the possibility to answer with predefined answers or voice. Something similar would be awesome.

@MarcusWolschon

This comment has been minimized.

Show comment
Hide comment
@MarcusWolschon

MarcusWolschon Apr 26, 2015

Member

At least offer the following:
single new mail: mark as reat, archive, spam, delete
multiple new mails: list individual mails, mark all as reat, archive all

optional:
single new mail: answer with a user defined text (default value translated of cause) e.g. "I'm got your message but I'm not on the desktop righ now. I'll get back to you later."

I'm willing to work on this feature but I'm in the middle of moving and changing jobs.

Member

MarcusWolschon commented Apr 26, 2015

At least offer the following:
single new mail: mark as reat, archive, spam, delete
multiple new mails: list individual mails, mark all as reat, archive all

optional:
single new mail: answer with a user defined text (default value translated of cause) e.g. "I'm got your message but I'm not on the desktop righ now. I'll get back to you later."

I'm willing to work on this feature but I'm in the middle of moving and changing jobs.

@cketti cketti added the enhancement label Apr 28, 2015

MarcusWolschon added a commit that referenced this issue Apr 28, 2015

Some first android wear support for enhancement
#619  "Add android wear support"

No reply with voice yet (as requested in the ticket).
No user-configurable actions yet, just delete+archive+spam
No stacked notification for multiple messages yet.

@MarcusWolschon MarcusWolschon self-assigned this Apr 28, 2015

@ashh87

This comment has been minimized.

Show comment
Hide comment
@ashh87

ashh87 Apr 30, 2015

Pebble too! Although there is something on Play store for this already, I don't know how good it is and I don't think it's open source.

ashh87 commented Apr 30, 2015

Pebble too! Although there is something on Play store for this already, I don't know how good it is and I don't think it's open source.

MarcusWolschon added a commit that referenced this issue May 2, 2015

Support for stacked notifications in
#619  "Add android wear support"

No reply with voice yet (as requested in the ticket).
No user-configurable actions yet, just delete+archive+spam
@MarcusWolschon

This comment has been minimized.

Show comment
Hide comment
@MarcusWolschon

MarcusWolschon May 2, 2015

Member

Using branch
https://github.com/k9mail/k-9/tree/issue-619_AndroidWearSupport
to develop this issue until it is ready for a pull request.

Member

MarcusWolschon commented May 2, 2015

Using branch
https://github.com/k9mail/k-9/tree/issue-619_AndroidWearSupport
to develop this issue until it is ready for a pull request.

MarcusWolschon added a commit that referenced this issue May 12, 2015

#619 "Add android wear support"
added EXTRA_MESSAGE_ID to simplify closing the correct notification and make things more robust.

MarcusWolschon added a commit that referenced this issue May 12, 2015

#619 "Add android wear support"
cleanup, comments and 1 missing intent-extra.

MarcusWolschon added a commit that referenced this issue May 23, 2015

#619 "Add android wear support"
Fixed wrong icon.
Summary Notification is phone-only again because it is often clicked when the user intents to do an action only on a single mail.

MarcusWolschon added a commit that referenced this issue Jun 10, 2015

MarcusWolschon added a commit that referenced this issue Jun 10, 2015

vt0r referenced this issue in vt0r/k-9 Jun 10, 2015

vt0r referenced this issue in vt0r/k-9 Jun 10, 2015

vt0r referenced this issue in vt0r/k-9 Jun 10, 2015

@Protonus

This comment has been minimized.

Show comment
Hide comment
@Protonus

Protonus Jun 24, 2015

I would love to see this, as it would allow me to mark as read / delete / mark as spam from my Pebble watch! Is this available in an alpha or beta build of K9?

Protonus commented Jun 24, 2015

I would love to see this, as it would allow me to mark as read / delete / mark as spam from my Pebble watch! Is this available in an alpha or beta build of K9?

@MarcusWolschon

This comment has been minimized.

Show comment
Hide comment
@MarcusWolschon

MarcusWolschon Jun 25, 2015

Member

#665 hasn't been merged yet.

Member

MarcusWolschon commented Jun 25, 2015

#665 hasn't been merged yet.

@Protonus

This comment has been minimized.

Show comment
Hide comment
@Protonus

Protonus Jun 25, 2015

@MarcusWolschon - can it be merged into an Alpha build so I can start using it? I see there are both Alpha and Beta channels. I would love to start using this!

Protonus commented Jun 25, 2015

@MarcusWolschon - can it be merged into an Alpha build so I can start using it? I see there are both Alpha and Beta channels. I would love to start using this!

@cketti

This comment has been minimized.

Show comment
Hide comment
@cketti

cketti Jun 25, 2015

Member

@Protonus If you don't want to build it yourself you have to wait until it is ready and released. Spamming all available channels isn't going to help.

Member

cketti commented Jun 25, 2015

@Protonus If you don't want to build it yourself you have to wait until it is ready and released. Spamming all available channels isn't going to help.

MarcusWolschon added a commit that referenced this issue Jul 5, 2015

#619 "Add android wear support"
removed dontCancel
update (stacked) notifications after a notification action was invoked

MarcusWolschon added a commit that referenced this issue Jul 5, 2015

#619 "Add android wear support"
removed dontCancel
update (stacked) notifications after a notification action was invoked
merged upstream changes

MarcusWolschon added a commit that referenced this issue Jul 7, 2015

#619 "Add android wear support"
* updating notifications after acting on one of many messages fixed.

* still debugging stacked notifications (deleting second of 2 messages) . CONTAINS DEBUGGING CODE

MarcusWolschon added a commit that referenced this issue Jul 14, 2015

#619 "Add android wear support"
* debugged stacked notifications (PendingIntents would reference wrong messages due to wrong handling of the PendingIntent mechanism)
* fixed dismissing notifications

MarcusWolschon added a commit that referenced this issue Jul 14, 2015

#619 "Add android wear support"
cleaned up debugging code
@allo-

This comment has been minimized.

Show comment
Hide comment
@allo-

allo- Aug 2, 2015

pebble already works, if you install the alternative notification center. This allows for the buttons in the android notifications as well as closing notifications on the phone with the watch.

allo- commented Aug 2, 2015

pebble already works, if you install the alternative notification center. This allows for the buttons in the android notifications as well as closing notifications on the phone with the watch.

@MarcusWolschon

This comment has been minimized.

Show comment
Hide comment
@MarcusWolschon

MarcusWolschon Aug 3, 2015

Member

But you get no stacked notifications to act on the individual messages without the merge of the #665 branch,
I'm just waiting for anyone to give the thumbs up to merge the pull request.

Member

MarcusWolschon commented Aug 3, 2015

But you get no stacked notifications to act on the individual messages without the merge of the #665 branch,
I'm just waiting for anyone to give the thumbs up to merge the pull request.

@Protonus

This comment has been minimized.

Show comment
Hide comment
@Protonus

Protonus Aug 3, 2015

@allo- I appreciate the suggestion,but it won't solve this. Notification Center is very complicated, interferes with other Pebble watch apps due to it's heavy reliance on Pebble's app message protocol, and is generally rather buggy due to what it's trying to accomplish. I much prefer stock notification support - which already gives me dismiss and other buttons, etc. What I want is actual Android Wear support for k-9 which will allow further actions on the watch, as this thread and #665 discuss.

@MarcusWolschon do you have a build / APK available of your branch, which supports either this or #665?

Protonus commented Aug 3, 2015

@allo- I appreciate the suggestion,but it won't solve this. Notification Center is very complicated, interferes with other Pebble watch apps due to it's heavy reliance on Pebble's app message protocol, and is generally rather buggy due to what it's trying to accomplish. I much prefer stock notification support - which already gives me dismiss and other buttons, etc. What I want is actual Android Wear support for k-9 which will allow further actions on the watch, as this thread and #665 discuss.

@MarcusWolschon do you have a build / APK available of your branch, which supports either this or #665?

@MarcusWolschon

This comment has been minimized.

Show comment
Hide comment
@MarcusWolschon

MarcusWolschon Aug 4, 2015

Member

I have an APK as I'm beta-testing the functionality for a few month now but it's signed by me.
So you would have to deinstall K-9, loose updates via Google Play and set up everything again.
It's just for development.

Member

MarcusWolschon commented Aug 4, 2015

I have an APK as I'm beta-testing the functionality for a few month now but it's signed by me.
So you would have to deinstall K-9, loose updates via Google Play and set up everything again.
It's just for development.

@MarcusWolschon

This comment has been minimized.

Show comment
Hide comment
@MarcusWolschon

MarcusWolschon Sep 15, 2015

Member

I'm back.
Aparently nobody has done anything in the last 1.5 month despite Wear-support being a popular requested feature.
What about simply merging this while we still can?
At some point it will just become one hell of a merge-conflict and nobody will want to do that at all anymore.

Member

MarcusWolschon commented Sep 15, 2015

I'm back.
Aparently nobody has done anything in the last 1.5 month despite Wear-support being a popular requested feature.
What about simply merging this while we still can?
At some point it will just become one hell of a merge-conflict and nobody will want to do that at all anymore.

@Protonus

This comment has been minimized.

Show comment
Hide comment
@Protonus

Protonus Sep 15, 2015

@MarcusWolschon Per your last message, can you link me to your APK or send me a copy? I'd rather use yours than wait any longer as it seems like this isn't going to get merged into even Alpha any time soon. I can backup my original config in Titanium and restore it once I see that this branch has been merged.

K-9 is one of my only apps I get notifications from on my watch, that doesn't support Android Wear yet, and it's one of the apps I'd use it most with.

Protonus commented Sep 15, 2015

@MarcusWolschon Per your last message, can you link me to your APK or send me a copy? I'd rather use yours than wait any longer as it seems like this isn't going to get merged into even Alpha any time soon. I can backup my original config in Titanium and restore it once I see that this branch has been merged.

K-9 is one of my only apps I get notifications from on my watch, that doesn't support Android Wear yet, and it's one of the apps I'd use it most with.

@MarcusWolschon

This comment has been minimized.

Show comment
Hide comment
@MarcusWolschon

MarcusWolschon Sep 16, 2015

Member

A cleaned up version has just been merged.

On Tue, Sep 15, 2015 at 4:15 PM, Protonus notifications@github.com wrote:

@MarcusWolschon https://github.com/MarcusWolschon Per your last
message, can you link me to your APK or send me a copy? I'd rather use
yours than wait any longer as it seems like this isn't going to get merged
into even Alpha any time soon. I can backup my original config in Titanium
and restore it once I see that this branch has been merged.

K-9 is one of my only apps I get notifications from on my watch, that
doesn't support Android Wear yet, and it's one of the apps I'd use it most
with.


Reply to this email directly or view it on GitHub
#619 (comment).

Member

MarcusWolschon commented Sep 16, 2015

A cleaned up version has just been merged.

On Tue, Sep 15, 2015 at 4:15 PM, Protonus notifications@github.com wrote:

@MarcusWolschon https://github.com/MarcusWolschon Per your last
message, can you link me to your APK or send me a copy? I'd rather use
yours than wait any longer as it seems like this isn't going to get merged
into even Alpha any time soon. I can backup my original config in Titanium
and restore it once I see that this branch has been merged.

K-9 is one of my only apps I get notifications from on my watch, that
doesn't support Android Wear yet, and it's one of the apps I'd use it most
with.


Reply to this email directly or view it on GitHub
#619 (comment).

@cketti cketti closed this in e76c489 Sep 27, 2015

@baudtack

This comment has been minimized.

Show comment
Hide comment
@baudtack

baudtack Sep 28, 2015

This patch doesn't provide a delete option. I get an email, and the notification on my pebble allows me to dismiss it, reply, mark read, archive, or mark as spam but not just delete a message.

baudtack commented Sep 28, 2015

This patch doesn't provide a delete option. I get an email, and the notification on my pebble allows me to dismiss it, reply, mark read, archive, or mark as spam but not just delete a message.

@cketti

This comment has been minimized.

Show comment
Hide comment
@cketti

cketti Sep 28, 2015

Member

Delete should show up when you disable 'Global settings > Interaction > Confirm actions > Delete (from notification)'

Member

cketti commented Sep 28, 2015

Delete should show up when you disable 'Global settings > Interaction > Confirm actions > Delete (from notification)'

@baudtack

This comment has been minimized.

Show comment
Hide comment
@baudtack

baudtack Sep 28, 2015

Ah. Thanks.

baudtack commented Sep 28, 2015

Ah. Thanks.

@ifuchs

This comment has been minimized.

Show comment
Hide comment
@ifuchs

ifuchs Nov 17, 2015

I have not beenable to get Delete to show up for K9 mail on Pebble. I tried changing Global settings per @cketti with no luck. Is there another trick to get this working?

ifuchs commented Nov 17, 2015

I have not beenable to get Delete to show up for K9 mail on Pebble. I tried changing Global settings per @cketti with no luck. Is there another trick to get this working?

@gappleto97

This comment has been minimized.

Show comment
Hide comment
@gappleto97

gappleto97 Dec 11, 2015

I'm also unable to get any actions besides "Dismiss" and "Open on Phone" to show up.

gappleto97 commented Dec 11, 2015

I'm also unable to get any actions besides "Dismiss" and "Open on Phone" to show up.

@baudtack

This comment has been minimized.

Show comment
Hide comment
@baudtack

baudtack Dec 11, 2015

Have you installed the Android Wear app? You don't need to run it just install it.

baudtack commented Dec 11, 2015

Have you installed the Android Wear app? You don't need to run it just install it.

@gappleto97

This comment has been minimized.

Show comment
Hide comment
@gappleto97

gappleto97 Dec 11, 2015

I did. I also have developer mode and accessibility access turned on for Pebble Time, to see if that made a difference.

gappleto97 commented Dec 11, 2015

I did. I also have developer mode and accessibility access turned on for Pebble Time, to see if that made a difference.

@gappleto97

This comment has been minimized.

Show comment
Hide comment
@gappleto97

gappleto97 Dec 12, 2015

Any idea what may be causing this? I'm happy to help any way I can.

gappleto97 commented Dec 12, 2015

Any idea what may be causing this? I'm happy to help any way I can.

@cketti

This comment has been minimized.

Show comment
Hide comment
@cketti

cketti Dec 12, 2015

Member

@gappleto97 Please use the mailing list for support requests. Once you're confident it is indeed a bug in K-9 Mail please create a new issue using the issue template.

PS: The new Android Wear code is not in any official builds yet. You have to create your own build from master (which is currently not very stable).

Member

cketti commented Dec 12, 2015

@gappleto97 Please use the mailing list for support requests. Once you're confident it is indeed a bug in K-9 Mail please create a new issue using the issue template.

PS: The new Android Wear code is not in any official builds yet. You have to create your own build from master (which is currently not very stable).

@MarcusWolschon

This comment has been minimized.

Show comment
Hide comment
@MarcusWolschon

MarcusWolschon Feb 24, 2016

Member

it's not in the 5008 release (at least the entire notification package is not in the 5008-rc1 branch).

Member

MarcusWolschon commented Feb 24, 2016

it's not in the 5008 release (at least the entire notification package is not in the 5008-rc1 branch).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment