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

Extract profile feature #10

Open
brh55 opened this issue Jul 5, 2017 · 1 comment
Open

Extract profile feature #10

brh55 opened this issue Jul 5, 2017 · 1 comment

Comments

@brh55
Copy link
Member

brh55 commented Jul 5, 2017

@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
Copy link
Member

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.

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

No branches or pull requests

2 participants