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

Migrate "threaded conversations" module to community-managed app #1

Open
pospi opened this issue Apr 15, 2019 · 0 comments
Open

Migrate "threaded conversations" module to community-managed app #1

pospi opened this issue Apr 15, 2019 · 0 comments
Assignees

Comments

@pospi
Copy link
Member

pospi commented Apr 15, 2019

It would be great to take threaded-comments and bring it into the Holochain org as was done for the file upload module that @RyanHow started. Note the rename from "comments" to "conversations" as I think this is a more generic name.

Would also like to discuss with @lorenjohnson & @nmitch17 / @forcize / @delroh to figure out what the best architecture for this module is. The work I was intending to do on it from here was:

  • Implement a total ordering of comments (probably via linked list), such that comments have a pointer back to the previous comment under each base entry and can be read in order. Some creative ordering / adjustment logic will be needed in the link management in order to resolve ordering conflicts resulting from network partitions.
  • Implement update capability:
    • add "last edited" field
    • implement reading of previous versions
    • implement author-based validation for update action (can only edit one's own posts)
    • implement time-based validation for edit action (can only edit within X minutes)
      • wire up to DNA configuration to make this configurable at deployment time
    • wire up DNA configuration to allow/disallow updates
  • package final module for consumption per Holochain/Rust guidelines
@pospi pospi changed the title Take on "threaded conversations" module Migrate "threaded conversations" module to community-managed app Apr 15, 2019
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