Skip to content
This repository has been archived by the owner on Oct 16, 2020. It is now read-only.

Custom scripts on new mail / no new mail event #37

Closed
wants to merge 4 commits into from
Closed

Custom scripts on new mail / no new mail event #37

wants to merge 4 commits into from

Conversation

SeniorSpielbergo
Copy link
Contributor

Hi,

I had a look at the links you gave me and implemented the possebility to add two scripts in the preferences.
The first one is run on new mail events and the other one if all mails are read.

If you want you can pull these changes, but you might want to have a look at the syntax because it is the first time I used java script.
You also might want to change the arrangement of the textboxes in the preferences. I could not find any better way to arrange them.

Kind regards,

David

@SeniorSpielbergo
Copy link
Contributor Author

I forgot to mention that it is essential that the script is executable (chmod +x option).

@foudfou
Copy link
Owner

foudfou commented May 24, 2012

I'll integrate your patch when I get a chance. Cheers.

@SeniorSpielbergo
Copy link
Contributor Author

Cheer!
I am discussing with somebody else about improvements on my idea (#33). Maybe you want to join in and add your thoughts if you have got the time.

@foudfou
Copy link
Owner

foudfou commented May 24, 2012

Hi, I integrated your patch and reworked it a bit (see new branch SeniorSpielbergo-master). In particular, I kept one trigger/script instead of two. But since the trigger now receives the new message count in argument, I guess you should be able to handle the branching in a single script (i.e. if arg == 0; then lightDown; else if arg > 0 then lightUp; else error). Tell me how it goes.

@SeniorSpielbergo
Copy link
Contributor Author

Hi, thanks I will try that next week. At the moment I am not a home. I will report back.

@SeniorSpielbergo
Copy link
Contributor Author

Hi I had a look at your changes. It looks like it is working and I think you can merge this in the master branch.
This way it is easier to work with scripts especially when others are really interested in the message count.
Thanks for your help on this.

@foudfou
Copy link
Owner

foudfou commented May 30, 2012

Ok. I just merged. Thank you.

@foudfou foudfou closed this May 30, 2012
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants