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

XOAUTH2 #53

Open
wants to merge 14 commits into
base: master
Choose a base branch
from
Open

XOAUTH2 #53

wants to merge 14 commits into from

Conversation

beckmx
Copy link

@beckmx beckmx commented Oct 26, 2013

I have changes to support XOAUTH2

adimircolen and others added 14 commits August 30, 2010 11:28
In GMail
 - Fix #labels on non-gmail accounts - stop assuming that the "[Gmail]"
   hierachy exists
 - Add #imap_xlist
   - Monkey patches XLIST support into Net::IMAP if it's missing.
   - Caches result until a new label is created
 - Add #inbox_label, #allmail_label, #spam_label, #trash_label,
   #drafts_label, #important_label, #starred_label
   - Uses XLIST to identify function of standard folders
 - Add #inbox, #allmail, #spam, #trash, #drafts, #important, #starred,
   #sent
   - Short cuts to known mailboxes
 - Add #normal_labels
   - Returns list of "normal" labels - ie. not sent, etc.
 - Add #label_of_type(type)
   - Looks up known mailbox

In GMail::Mailbox
 - In #emails
   - Add support for additional all IMAP search keywords via hash params
     to #emails().
   - Throw runtime error for unknown keys

In GMail::Message
 - Add #message_id
   - UIDs are different for the same message across different folders,
     so Message-ID header is the test for uniqueness
 - Add #has_label?(name)
   - Indicates whether message has given label applied
 - Add #archived?, #starred?, #sent?, #important?
   - Shortcuts to check whether the message has been archived, starred
     etc.
 - Add #labels - returns list of labels applied to message
   - Search all known folders for messages with this message-id
 - Add #remove_label(label) - deletes message from given mailbox/label
   - ie. delete message from named folder
In Gmail::Mailbox
 - Add support for :fetch key in #emails() to indicate the message
   bodies should be fetched

In Gmail::Mail
 - Add #loaded? to check whether message has already been downloaded
 - Add #set_body(body) to set the contents of a message
GMail::Message
- Add #envelope to store/read back imap envelope contents
- Add #message_id? to check whether we know the message id yet
- Add #subject, #from, #to to read values from envelope with body
  fallback

GMail::Mailbox
- Change behaviour - always fetch IMAP ENVELOPE for search results
  unless we have already loaded message
- Add GMail::Mailbox::MessageList - array like class for search results
- Add MessageList#with_label(label) - filter results
  - Searches label folder for messages in MessageList
@myobie
Copy link
Collaborator

myobie commented Mar 28, 2014

I am not sure if I'll be able to merge this. With so many changes and no updated specs it seems like it could be disastrous since I don't understand all of it. Do you want to work together to spec this out?

@beckmx
Copy link
Author

beckmx commented Mar 28, 2014

yeah sure, but it is not that complex, actually it is just an extra string I think is missing

@myobie
Copy link
Collaborator

myobie commented Mar 28, 2014

Oh, when I look at the diff here on github there are a lot of changes. Do you have a simpler example of what you need?

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.

5 participants