Show all a contact's email addresses in the output of ppl show #14

Closed
storge-xx opened this Issue Dec 28, 2012 · 6 comments

Projects

None yet

3 participants

@storge-xx

several people i know--myself included--have more than one email address.

my first thought was to just make multiple ppl for someone.

ppl add kevin "kevin@somewhere.net"
ppl org kevin "home"
ppl add kevin2 "kevin@elsewhere.net"
ppl org kevin2 "work"

it works well enough for me.

but the reason i went ahead and posted this, is that now that i see axelGschaider talking about nucky, i started thinking that i'd like being able to add more than one email to a contact--especially if they are ever able to be synced. i have some individual android gmail contacts with 3 emails and 3 phone numbers! I know that's outlier, but i think there are enough people with two or more emails and phone numbers that it's bound to come up at some point.
...but that seems like a fair amount of increase in complexity for you and ...well its just a request :)

i'm happy to report i have about 100 ppl contacts already. i just integrated my gmail with mutt in the yesterday, and i'm about to post this here and go look at goobook (for gmail contacts in mutt) and see where things overlap or are lacking. unfortunately i'm not much of a programmer, so i'm sort of looking at it from the geeky user perspective.

@axelGschaider

hi storge,

multiple email addresses and phone number appear to be on the http://ppladdressbook.org/development/roadmap/. Guess we'll see them rather soon.

Makes me think: some people have multiple postal addresses. Maybe that should be supported too?

@hnrysmth
ppl member

What version are you guys using? Check out the changelog: support for this has been available since 1.4.1. The latest version, 1.6.0, fixes a bug in it and adds the ability to remove email addresses using ppl email --delete.

$ ppl email john john@example.org
$ ppl email john john@example.com
$ ppl email john
john@example.org
john@example.com

One thing that's missing is that ppl show still only shows the first email address. But ppl email will show you the full list already.

there are enough people with two or more emails and phone numbers that it's bound to come up at some point.

Makes me think: some people have multiple postal addresses. Maybe that should be supported too?

Absolutely. This is definitely planned, don't worry! In fact, it was supported in the first little "quick and dirty" prototype that I built, before I threw it away to build a version that would actually be maintainable!

Supporting more than one email address, postal address, phone number (and so on) per contact is actually my number one priority. It's why I'm reluctant to drop everything for GMail sync while this fundamental deficiency in ppl's functionality is still there.

@storge-xx

thanks, and i had updated/tested before i commented above, but i had only tried ppl show and saw only one addy. cool!

@hnrysmth
ppl member

Okay then, in that case I'm glad I asked, because this shows that it was a dumb idea for me to add multiple email address support and then deliberately not add it to ppl show. Another lesson learned, at least.

I'm going to rename this issue as "Show all a contact's email addresses in the output of ppl show" and I'll close it once that's done.

@hnrysmth hnrysmth was assigned Dec 28, 2012
@hnrysmth hnrysmth added a commit that referenced this issue Dec 29, 2012
@hnrysmth hnrysmth Change the output format of ppl show substantially.
In part this change is motivated by issue #14, in which @storge pointed out that
the ppl show command only displays one of the contact's email addresses. This
gives the impression that only one can be stored per contact, so it needed
changing urgently.

Another reason for this change was that the output of ppl show was on the whole
quite an ugly, disorganised mess. This commit changes the rest of the formatting
somewhat so that hopefully it'll be a little more clear.
c8c54c0
@hnrysmth
ppl member

This is all better now in 1.7.0. Thanks for asking about this one. I knew I was being lazy at the time when I didn't update ppl show to handle multiple email addresses.

By the way @storge and @axelGschaider, thanks so much for all your help, feedback and patience so far! It's been invaluable for clearing out some of the inevitable "brand new project" suckiness!

@hnrysmth hnrysmth closed this Dec 29, 2012
@axelGschaider

You're welcome.
But I'd call it "brand new project awesomeness" :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment