search

crypti / ccoin

:part_alternation_mark: Convert and fetch the cost of over 1000 cryptocurrencies within your terminal, and save repeated queries
MIT License
23 stars 1 forks source link

Extract profile feature #10

Open brh55 opened 7 years ago

brh55 commented 7 years ago

@radiovisual I'm looking to implement the exact profile feature in the ccnews once I can figure out these encoding issues, but I think we can abstract that type of logic in it's own package.

Judging from the existing code, it looks possible. I'm thinking of an simplistic browser storage-like API.

set(profile, profileObject) get(profile, profileObject) delete(profile)

What are your thoughts? I can tackle this also on my free time, but just wanted to let you know.

radiovisual commented 7 years ago

Hey @brh55, so you mean you want to consolidate the profile management APIs between ccoin and ccnews by extracting the profile management for ccoin into it's own module so you can use it in ccnews? Sounds like a good plan to me! 👍

The API you suggested looks good.