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

change documentation to reflect that Imbox.messages() gets all messages in the inbox #128

Closed
zevaverbach opened this issue Jul 25, 2018 · 2 comments · Fixed by #134
Closed

Comments

@zevaverbach
Copy link
Contributor

zevaverbach commented Jul 25, 2018

The comment and var name in the docs strongly imply all_messages, but Imbox.messages() only returns a small subset of messages, whichever ones are in the inbox, not archived. Would you accept such a PR?

@zevaverbach
Copy link
Contributor Author

It might be good to also change the logging in Imbox.messages in this case to "Fetch list of messages from inbox." or something similar.

@martinrusev
Copy link
Owner

@zevaverbach Some verbosity in the logging and docs could be a good addition.

zevaverbach added a commit to zevaverbach/imbox that referenced this issue Jul 26, 2018
…folder is specified in the kwargs to Imbox.messages, the returned messages will be from the inbox. In the documentation this is accomplished exclusively by the var names. fixes martinrusev#128.
zevaverbach added a commit to zevaverbach/imbox that referenced this issue Jul 26, 2018
…Fixes martinrusev#124.  Added Pycharm directory to .gitignore.

Fixed var names in documentation of query keywords

moved Messages and Imbox to their own modules, imported Imbox.imbox into __init__.py and put it in __all__. fixes martinrusev#130.

clarified in documentation and Imbox.messages logging that, unless a folder is specified in the kwargs to Imbox.messages, the returned messages will be from the inbox.  In the documentation this is accomplished exclusively by the var names. fixes martinrusev#128.

amended `8df7d7c` to reflect manual changes made to `README.rst` in current master, but also added `inbox_` to several var names to make that explicit in the documentation.  Added flags to messages returned by `fetch_email_by_uid`, using the new function `parse_flags` in `parser.py`.  Fixes martinrusev#126.

added TODO back into query.py
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants