Skip to content
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

New mail hook #238

Closed
wants to merge 1 commit into from
Closed

New mail hook #238

wants to merge 1 commit into from

Conversation

krafczyk
Copy link
Contributor

@krafczyk krafczyk commented Sep 9, 2015

Second try at resolving #236.

@krafczyk
Copy link
Contributor Author

krafczyk commented Sep 9, 2015

Sorry about the last pull request.

I hope I've fixed all the problems this time.

@nicolas33
Copy link
Member

You can squash both patches. Please, update offlineimap.conf accordingly and mark this feature as experimental.
Thank you.

@krafczyk
Copy link
Contributor Author

krafczyk commented Sep 9, 2015

Where do I mark the feature as experimental?

Next to the entry in offlineimap.conf?

On Wed, Sep 9, 2015 at 8:55 PM, Nicolas Sebrecht notifications@github.com
wrote:

You can squash both patches. Please, update offlineimap.conf accordingly
and mark this feature as experimental.
Thank you.


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

@nicolas33
Copy link
Member

Yes, in offlineimap.conf. :-)

When new mail arrives, this hook is triggered, allowing the user to
play a sound, or launch a popup.

Signed-off-by: Matthew Krafczyk <krafczyk.matthew@gmail.com>
@krafczyk
Copy link
Contributor Author

Okay, I've squashed the commits, and added the relevant section to offlineimap.conf.

@nicolas33
Copy link
Member

Thank you ! Applied. :-)

@nicolas33 nicolas33 closed this Sep 11, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants