cdunn6754 / rereddit_frontend Goto Github PK

I have 'dangerouslySetInnerHtml' in the post and comment lists. I want to include that styling that is provided by the cool wysiwyg editor. This should be okay because any html typed by a user into the editor will be a part of a string and won't be rendered.

The issue is that the api can still be accessed outside of the frontend. There a consumer could submit a new comment or post with malicious html in the body. Not sure how to stop that.

Currently there is just a "coming soon" banner there. Its a little trickier than it seems because we need to
alter the backend as well. We need to add the post details of each comment to the data. That way the comments can be grouped together into a single entry and links to the post detail page of their post can be added.

This is related to this backend issue

Like on the real reddit site, rather than loading all of the posts for a subreddit at one time load pages of 20 or so and then attach a loader at the end of the list that when brought into the window triggers another fetch from the api of the next 20.

Related to cdunn6754/reReddit_backend#2 (comment)

Currently we store the two types of comments in disparate redux slices. I don't think that's necessary since we will never need both at the same time (caching may not be a bad idea down the road too). The idea would be to use the comments redux slice, that is currently only used for the trees of comments corresponding to the post detail view, to store the comments for the user profile view.

It's a little complicated because the post detail comments are nested by the backend and the user profile comments are just a list of comments that the user has made.

It may be best to use the nested comment serializer for both cases and specify the queryset, i.e. whether we want all comments related to a post or related to a user, with either separate views or POST/GET parameters.

Nesting the user profile comments would almost be nice because then the work of grouping them by post is done in the backend.

When you create a new comment and then reply to it before the list is reloaded the child will not
render properly. I think that we just need to be sure to add a children property to the comments when they
are first created. In fact that might be something that should be taken care of on the backend for
consistency.