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

Gmail DeleteMessage and GetMessages setseen #86

Open
paulpps opened this issue Jun 13, 2012 · 7 comments
Open

Gmail DeleteMessage and GetMessages setseen #86

paulpps opened this issue Jun 13, 2012 · 7 comments

Comments

@paulpps
Copy link

paulpps commented Jun 13, 2012

There appears to be an issue with the delete method for Gmail. The delete command removes the message from the inbox, but does not mark it as seen. By running two separate commands with \Seen and \Delete it looks like it does the trick.

Logic is inverted for setseen when calling GetMessages. Passing false will mark the message as seen while passing true will not.

@smiley22
Copy link

smiley22 commented Aug 7, 2012

Yea you're right, the logic for the setseen flag was inverted...

@filpen
Copy link
Contributor

filpen commented Jan 29, 2013

Is it possible to have this merged? I stumbled upon the same issue and fixed it in the same way, it is just a one-liner fix. If the problem is that the pull request is of considerable size and Andy didn't have time yet to review it as a whole I can submit a new pull request containing just that fix.

@andyedinborough
Copy link
Owner

@filpen, please do. Thanks!

@filpen
Copy link
Contributor

filpen commented Jan 29, 2013

@andyedinborough that was quick, thanks ;)

@andyedinborough
Copy link
Owner

;] I'm trying to get better.

@andyedinborough
Copy link
Owner

I could have sworn this issue was resolved a long time ago actually...

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

No branches or pull requests

5 participants
@andyedinborough @paulpps @smiley22 @filpen and others