Skip to content

Update lib.rs#7

Open
falyhery wants to merge 1 commit intoholochain-devcamp:masterfrom
falyhery:patch-1
Open

Update lib.rs#7
falyhery wants to merge 1 commit intoholochain-devcamp:masterfrom
falyhery:patch-1

Conversation

@falyhery
Copy link

  • Implement section entry definitions
  • Implement section CRUD methods
  • Implement content entry definitions
  • Implement content CRUD methods

- Implement section entry definitions
- Implement section CRUD methods 
- Implement content entry definitions 
- Implement content CRUD methods
@e-nastasia
Copy link
Member

All looks good to me!
I also have an update here on the code structure and how can be the same thing done a bit differently. I only realized it while already in the devcamp: I decided to move the get_latest_entry function implementation into the corresponding handlers section to have lib.rs be only an API definition that doesn't really contain any implementation details. This is just a matter of taste and it doesn't change anything in the code correctness. You can see what I meant in this commit.

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

Successfully merging this pull request may close these issues.

2 participants