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.

Sign up for GitHub

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

Jump to bottom

Make imap.MailboxStatus extendable #57

Closed
emersion opened this issue Sep 23, 2016 · 0 comments
Closed

Make imap.MailboxStatus extendable #57

emersion opened this issue Sep 23, 2016 · 0 comments
Labels
enhancement question
Milestone
v1

Comments

@emersion
Copy link
Owner

emersion commented Sep 23, 2016 •
edited
Loading

  • Make it a map? (like https://golang.org/pkg/net/mail/#Header)
    • Would require a lot of getters and setters :(
    • See 8d1e889
  • Use an interface?
    • Would require also getters and setters
  • Mix a struct and a map?
    • Keep current fields, ie. keep backwards compatibility
    • Add a Items map[string]interface{} field which contains all items and their values
    • See f051cce
  • Find a better solution?
@emersion emersion added this to the v1 milestone Sep 23, 2016
@emersion emersion changed the title Change mailbox status to be a map Make mailbox status extendable Oct 2, 2016
@emersion emersion changed the title Make mailbox status extendable Make imap.MailboxStatus extendable Oct 16, 2016
@emersion emersion mentioned this issue Oct 16, 2016
Closed
@emersion emersion mentioned this issue Oct 16, 2016
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement question
Projects
None yet
Milestone
v1
Development

No branches or pull requests
1 participant
@emersion