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

Add Likes & Comments Option For Posts #111

Open
NVombat opened this issue Feb 22, 2022 · 16 comments · May be fixed by #144
Open

Add Likes & Comments Option For Posts #111

NVombat opened this issue Feb 22, 2022 · 16 comments · May be fixed by #144
Assignees
Labels
backend For backend related issues django Django related development enhancement New feature or request

Comments

@NVombat
Copy link
Owner

NVombat commented Feb 22, 2022

Detailed description

As of now the posts are displayed on the POSTS page and the user can see other user's posts. Once the front end develops a custom video/audio player to show the posts, we want to add the feature of having Likes & Comments for each post. The comments may be filtered (No swear words etc) and can also have a nested structure where replies can be shown (How it is shown on Instagram and Facebook).

In addition to this the admin should be allowed to delete any comments if they are not detected by the filter. That way there is a method to regulate the content.

Context

This will improve the user experience and make it more than just a platform for sharing media by enabling users to interact rather than just post and view other user's posts.

Possible implementation

Add additional columns to the Database for a count of likes & a store for comments which can be updated as and when the user receives any likes or comments. The comments can be stored in a list or dictionary nested within the post data

Add a functionality for the admins to delete the comments which is a simple delete operation in the database.

For Filtering of Comments, we can either use a simple model to detect bad words from a list of predefined bad words or even have a machine learning model which filters comments.

@NVombat NVombat added the enhancement New feature or request label Feb 22, 2022
@github-actions
Copy link

Hello, Thank You for raising this Issue. Our Team will look into the issue and discuss with you.

@NVombat NVombat changed the title Add Likes & Comments Option To Posts Add Likes & Comments Option For Posts Feb 22, 2022
@NVombat NVombat added backend For backend related issues django Django related development labels Feb 22, 2022
@Vrbitko
Copy link

Vrbitko commented Oct 4, 2022

Hello, is this issue still available? Thanks

@NVombat
Copy link
Owner Author

NVombat commented Oct 5, 2022

Hello, is this issue still available? Thanks

Yes the issue is available. Let me know if you wanna take it up

@Vrbitko
Copy link

Vrbitko commented Oct 5, 2022

Yes sure, will take it.

@NVombat
Copy link
Owner Author

NVombat commented Oct 6, 2022

Yes sure, will take it.

Perfect, if you need any assistance feel free to reach out... Will help you wherever I can

@NVombat
Copy link
Owner Author

NVombat commented Oct 19, 2022 via email

@Vrbitko
Copy link

Vrbitko commented Oct 19, 2022

Hey! Im on BE but also tried to run FE. Will update you during the week w databases diagrams so you will agree on changes about coments and likes. Also had problems w running the project. Have you been thinking w updating readme/docker? Currently I've already checked code and I am ready to start.

@Vrbitko
Copy link

Vrbitko commented Oct 19, 2022

The problem with envs - currently solved, if any issue will occure, will write you, thx

@NVombat
Copy link
Owner Author

NVombat commented Oct 20, 2022 via email

@Vrbitko
Copy link

Vrbitko commented Oct 20, 2022

So, i would suggest something like that:
image

  1. suggestion: From my point of view its better NOT TO USE DICTS as you suggested. Would love to implement it using https://pydantic-docs.helpmanual.io/ and create own models so there will be implemented better typing for the future.
  2. suggestion: i saw that you are using email of user as foreign key. Is there reason for that? Cannot you just use PID of user? At the photo i used that pid as FK.

Thank you

@NVombat
Copy link
Owner Author

NVombat commented Oct 21, 2022 via email

@Vrbitko
Copy link

Vrbitko commented Oct 21, 2022

Yes sure it is! Which platform for calls do you prefer? Can we meet tomorrow 16:00 CET?

@NVombat
Copy link
Owner Author

NVombat commented Oct 22, 2022 via email

@Vrbitko
Copy link

Vrbitko commented Oct 23, 2022

@NVombat I've sent you invite to linkedin, we can agree on call there. It will be faster.

@NVombat
Copy link
Owner Author

NVombat commented Oct 23, 2022 via email

@NVombat
Copy link
Owner Author

NVombat commented Nov 2, 2022

Hey, I wont be able to share any credentials but I have added a little setup part in the server/README.md file so you can set up your own env

@Vrbitko Vrbitko linked a pull request Nov 8, 2022 that will close this issue
10 tasks
@NVombat NVombat linked a pull request Nov 8, 2022 that will close this issue
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend For backend related issues django Django related development enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants