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

Refactor Append-only Example from UUID to use CID #22

Open
RobStallion opened this issue Jan 31, 2019 · 4 comments
Open

Refactor Append-only Example from UUID to use CID #22

RobStallion opened this issue Jan 31, 2019 · 4 comments
Assignees
Labels
enhancement New feature or request help wanted Extra attention is needed

Comments

@RobStallion
Copy link
Member

@nelsonic based on the following comment you made in here...

The work to make Phoenix append-only is "on-going". Once CID is complete we will need to integrate it into our Example: https://github.com/dwyl/phoenix-ecto-append-only-log-example and then add it to Alog (which will need to be re-worked to make the log "generic" ... tbd.)

Would I be correct in assuming that now that CID has been published, updating this example is the next priority (one of the next priorities) on the list?

@RobStallion RobStallion added the question Further information is requested label Jan 31, 2019
@nelsonic
Copy link
Member

@RobStallion thanks for opening this issue. I was about to create it ... 👍
Please estimate and proceed. thanks. ✨

@nelsonic nelsonic added enhancement New feature or request and removed question Further information is requested labels Jan 31, 2019
@nelsonic nelsonic removed their assignment Jan 31, 2019
@nelsonic nelsonic changed the title High priority? Refactor Append-only Example from UUID to use CID Jan 31, 2019
@RobStallion RobStallion self-assigned this Jan 31, 2019
@RobStallion
Copy link
Member Author

RobStallion commented Feb 1, 2019

@nelsonic I have had a brief read through the current example and the alog code and have a few question/thoughts on how things might/will change when we use CIDs.

I am going to open a separate issue so that it is easier for people to comment with their thoughts for how that particular function will work.

@nelsonic
Copy link
Member

nelsonic commented Feb 1, 2019

@RobStallion sounds good. We can do another Zoom Call later today (or Monday) to go through it. 👍

@RobStallion
Copy link
Member Author

@nelsonic Yeah I'd be up for that. I'll write up my thoughts in issues first though so that others can comment and maybe join if that makes sense 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants