Feat: Like feature is not implemented properly.

[Sanchitbajaj02]

What feature?

The code for like feature is implemented but it is not working properly on the server. A PoC is required before going further.

Add screenshots

No response

Record

• I agree to follow this project's Code of Conduct
• I want to work on this issue

[github-actions]

Congratulations, @Sanchitbajaj02! 🎉 Thank you for creating the issue. Your contribution is greatly appreciated and we look forward to working with you to resolve the issue. Keep up the great work!

'We will promptly review your changes and offer feedback. Keep up the excellent work!
Kindly remember to check our Contributing Guidelines'

[Archit5655]

I would like to take on this issue

[Sanchitbajaj02]

Sure, please go ahead. If you need anything, just let me know. Also, we need to discuss about its performance. So, before implementing anything, just ask here or in discord

[sumanjeet0012]

I want to work on this issue.
assign me

[Sanchitbajaj02]

@sumanjeet0012 assigned to you

[Jenis07]

Can you assign me this issue under IWOC?

[Sanchitbajaj02]

@Jenis07 sure. Assigned this to you. Make sure to update your progress and if you have any doubt, just let me know.

[IrfanshaikH-7]

Can I look into it ...@Sanchitbajaj02

[Jenis07]

is there any steps to setup the project in the localhost?

[IrfanshaikH-7]

Yes checkout the readme - it explained everything

[IrfanshaikH-7]

is there any steps to setup the project in the localhost?

And you have to setup appwrite instance too .. if you follow the readme doc , you shouldn't have any issues with seting up..

[Manishak798]

@Jenis07 have you started working on this ?

[Sanchitbajaj02]

@Jenis07 If you don't respond within 24 hours, I will unassign the issue

[Jenis07]

Yes sir I have started working on this and I have made some logic for that.I will look this by today.

…

On Mon, Jan 29, 2024, 23:24 Sanchit Bajaj ***@***.***> wrote: @Jenis07 <https://github.com/Jenis07> If you don't respond within 24 hours, I will unassign the issue — Reply to this email directly, view it on GitHub <#92 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AX5QX7C74RBMWKJNTSLSANLYQ7O6DAVCNFSM6AAAAABBCJ4HLCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJVGI3DOOBQGU> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[Sanchitbajaj02]

@Jenis07 any updates?

[ArtiGaund]

So, there are User, Post, and like collections. When the user clicks on the like button, => if like is already enabled (checking whether post id with user id is present in the Like collections), it will delete that entry from the like collection and decrease the count of post ids in Post collection. Else will increase the count of post IDs in Post Collection and add entry in Like collection with post IDs and user IDs.

[Sanchitbajaj02]

So, there are User, Post, and like collections. When the user clicks on the like button, => if like is already enabled (checking whether post id with user id is present in the Like collections), it will delete that entry from the like collection and decrease the count of post ids in Post collection. Else will increase the count of post IDs in Post Collection and add entry in Like collection with post IDs and user IDs.

Yes, that's the aim of this functionality

[ArtiGaund]

I will try to do it.

[ArtiGaund]

login feature is not working; yesterday it was working when I created account after setting up the project, but today when I am trying to login, it is showing this error. Even after I modified the password in the database, it still shows this error. I am not able to see the post page.

[ArtiGaund]

whats the timelimit for the assignment?

[Sanchitbajaj02]

whats the timelimit for the assignment?

There is no as such time limit, if you need time, you can take it. Just keep me updated with the progress. As for the login issue, what you can do is peek into the appwrite and see whether in the auth you are verified or not. If yes, go to db and see whether an entry is created with your username. Let me know here or on discord about your findings...

[ArtiGaund]

For login, I am registering and deleting accounts daily, as I have appwriteDB in my appwrite console. Through registration, I am able to get a login automatically and can view the posts and work on them.

[Sanchitbajaj02]

For login, I am registering and deleting accounts daily, as I have appwriteDB in my appwrite console. Through registration, I am able to get a login automatically and can view the posts and work on them.

Actually, you don't need to delete daily. What you can do is find out what part is giving the error in the component.

[ArtiGaund]

I will try to find it.

[ArtiGaund]

Resolved you registeration and login problem.

[Sanchitbajaj02]

Resolved you registeration and login problem.

Can you address the issue and your findings in #300

[ArtiGaund]

as the appwrite version is fixed in the project, I cannot use upgrade version methods? bz they will not work.

[Sanchitbajaj02]

as the appwrite version is fixed in the project, I cannot use upgrade version methods? bz they will not work.

Does your implementation require it to be on an upgraded version?

[ArtiGaund]

as the appwrite version is fixed in the project, I cannot use upgrade version methods? bz they will not work.

Does your implementation require it to be on an upgraded version?

For performing like and dislike, I need to check whether the user has liked the post or not. To check that entry, their is a query.and() method that checks multiple queries together. But Query.and() is in the upgraded version of AppWrite. The assignment is not yet complete, but bugs are there. That's why I am asking; I will change that and use a different method to do it.

[Sanchitbajaj02]

But Query.and() is in the upgraded version of AppWrite. The assignment is not yet complete, but bugs are there. That's why I am asking; I will change that and use a different method to do it.

There will be multiple changes like that login issue? if you want we can switch to higher variant but its just that we need to cater those changes

[ArtiGaund]

Login issue is resolved now?

[Sanchitbajaj02]

Login issue is resolved now?

That's not the issue. If it was working fine on version 13 then it isn't a bug. Since it is not recommended to upgrade the version when I provided the lock functionality

[ArtiGaund]

I will complete the like functionality first, which was assigned to me. Then we will pull upstream and check the error for login and registration.

[ArtiGaund]

[Sanchitbajaj02]

Check whether the proper type is declared for onLikeClick. If you face any issue, please reach out to me on discord

[vinayak-vohra]

@Sanchitbajaj02 may I work on this issue?

[Sanchitbajaj02]

@Sanchitbajaj02 may I work on this issue?

yeah sure

[vinayak-vohra]

@Sanchitbajaj02 the posts collection has a likesCount attribute of integer type which would only tell the number of likes.
There is no way of knowing whether the current user has liked the post or not, which is needed to implement the like and unlike functionality. There is also a likes collection which is never used. I've thought of a few ways to tackle this issue:

1. Change likesCount(integer) to likedBy(array of userIds):
   • Like count would be the size of likedBy array.
   • For Like/Unlike we can check the array for current userId. If present, splice the array and update document (unlike). Else push current userId into the array (like).
   • In this approach, userIds of all users who liked the post would be sent.
2. Utilizing the likes collection:
   • We can fetch all the docs with userId as current user and convert it to an array of postIds (likedPosts) since userId will be same in all.
   • Liked status for UI can be checked using likedPosts.includes(postId).
   • Like/Unlike would be pushing/splicing from likedPost array.
   • This approach would require 2 API calls: updating count in document of posts collection and creating/deleting document in likes collection.

Please suggest which approach to implement, or if you'd like to suggest any other approach.

[vinayak-vohra]

@Sanchitbajaj02 I've implemented the like feature, but it requires some changes in the structure of likes collection. So, I need to discuss a few things related to this. Please reply when you are available.

palettegram-like.mp4

[Sanchitbajaj02]

@Sanchitbajaj02 I've implemented the like feature, but it requires some changes in the structure of likes collection. So, I need to discuss a few things related to this. Please reply when you are available.

My availability is and will be a bit low. I know it needs to be structured. You can DM me on discord about your idea and we can discuss over there