-
Notifications
You must be signed in to change notification settings - Fork 6
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
Nested comments #109
base: master
Are you sure you want to change the base?
Nested comments #109
Conversation
…in commentlist to include editorbox if applicable
…o publish comment after in ReplyTo Editor box, fix the toggling to only show/hide the text editor box for corresponding ReplyTo, and hide all editor boxes at initializaiton
…entire comments list; Toggle for ReplyTo button to show/hide Editor Box
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/bitproject/bit-next/mvlsdvmzx |
<Comment | ||
key={comment.id} | ||
actions={[<span key="comment-nested-reply-to">Reply to</span>]} | ||
actions= {[ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When the comment is made the editor should close
What did you implement/accomplish in this pull request?
Give a short summary of what you completed in the pull request.
This is for issue #96
This allows logged in users to post comments under existing comments by clicking the Reply To option.
The users who are not logged in will receive a message that reminds them to log in.
Backend Checklist
flask test
Frontend Checklist
npm run build
does not create any errors