Comments First Implementation #788

Closed
opened 2017-11-26 23:57:07 +01:00 by ghost · 0 comments
ghost commented 2017-11-26 23:57:07 +01:00 (Migrated from github.com)

"Edit: After our recent meeting, the best way to do this seems to be to add a message to to the current support feature that gets displayed on the page."

-- Liam

The Issue

Commenting first run

Parameters

  1. Comments cost default price (below notification threshold)
  • Payment on blockchain mirrors "tip support" button, appears as "tip" or "comment tip" in History
  1. Comments sort by
  • IF has most current time stamped child comment, THEN display on top
  • OR is only Parent with Child, THEN display on top
  • ELSE display most current time stamp parent comment on top
  1. Comment Owner = Default Channel/Identity of Commenter's app
  2. Comment Owner's name links to their channel URL

To Add Later

  1. Channel owner toggles comment price
  2. Upvotes(?)
  3. Pinned comments by creator

Expected behaviour

Tell us what should happen

Actual behaviour

Tell us what happens instead

System Configuration

  • LBRY Daemon version:
  • LBRY App version:
  • LBRY Installation ID:
  • Operating system:

Anything Else

Screenshots

"**Edit:** After our recent meeting, the best way to do this seems to be to add a message to to the current support feature that gets displayed on the page." -- Liam ## The Issue Commenting first run ### Parameters 1. Comments cost default price (below notification threshold) - Payment on blockchain mirrors "tip support" button, appears as "tip" or "comment tip" in History 2. Comments sort by - IF has most current time stamped child comment, THEN display on top - OR is only Parent with Child, THEN display on top - ELSE display most current time stamp parent comment on top 3. [Comment Owner = Default Channel/Identity of Commenter's app](https://github.com/lbryio/lbry-app/issues/787) 4. Comment Owner's name links to their channel URL ### To Add Later 1. Channel owner toggles comment price 2. Upvotes(?) 3. Pinned comments by creator ### Expected behaviour Tell us what should happen ### Actual behaviour Tell us what happens instead ## System Configuration <!-- For the app, this info is in the About section at the bottom of the Help page. You can include a screenshot instead of typing it out --> <!-- For the daemon, run: curl 'http://localhost:5279' --data '{"method":"version"}' and include the full output --> - LBRY Daemon version: - LBRY App version: - LBRY Installation ID: - Operating system: ## Anything Else <!-- Include anything else that does not fit into the above sections --> ## Screenshots <!-- If a screenshot would help explain the bug, please include one or two here -->
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: LBRYCommunity/lbry-desktop#788
No description provided.