Skip to content

Commit

Permalink
Create first draft of comment object
Browse files Browse the repository at this point in the history
The vision of comments is that users either quote something directly
from the abstract (optional `quotedSnippet`) via highlighting a passage,
and/or making general comments (`commenterInput`). Submitters will have
commenter names that are not fellow submitters made anonymous.

Top level comments (TLC's) can be resolved or not (defaults to `false`),
and comments on TLC's may or may not need to be resolved but for now
they are optionally resolvable with the idea that they only contribute
to the TLC being resolved. Resolved comments can either be hidden, or
perhaps hidden from the normal comment space and shown in some sort of
`Resolved` area like a dropdown for posterity, or crossed out in the
normal comment sidebar? Perhaps if the TLC's exigence is argued to be
relevant again then the Resolved TLC could be toggled to be an active,
unresolved comment?

All TLC's and subcomments should be timestamped.

Issue #147 (Create comment object)
Epic #217 (Comment on Abstracts)
Comments are subset of: Epic #131 (Submission Card View)
  • Loading branch information
michael-small committed Nov 30, 2019
1 parent bb7eb13 commit 5ff024e
Show file tree
Hide file tree
Showing 2 changed files with 12 additions and 1 deletion.
1 change: 0 additions & 1 deletion client/src/app/newSubmission/submission.ts
Original file line number Diff line number Diff line change
@@ -1,4 +1,3 @@

export interface Submission {
_id?: {
$oid: string
Expand Down
12 changes: 12 additions & 0 deletions client/src/app/submissionView/comment.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1,12 @@
import {User} from "../user";

export interface Comment {
_id?: {
$oid: string
};
commenters: Array<User>;
quotedSnippet?: string; //generated by highlighting abstract... may not always be needed
commenterInput: string;
resolved?: boolean; //Top level comments (TLC's) resolvable, comments on TLC resolved via TLC resolution
timestamp: string;
}

0 comments on commit 5ff024e

Please sign in to comment.