Marc Seitz
Marc Seitz
/award 400 thank you so much for the following and sharing with your friends
https://github.com/mfts/papermark/pull/1667#discussion_r2100778206 @Aashish-Upadhyay-101 can you also add the suggestion about the memoization of the foldercount.
> Would it make sense to add stored fields (e.g., totalDocuments, totalChildFolders) on the folder model and update them? > What do you think? That's a good suggestions. I kinda...
Good catch @avikalpg.
@AnasKhan321 - please sign the CLA as indicated above - merge current main branch and solve any conflicts
@AnasKhan321 please see my last message above.
Thanks for your effort. I didn't hear back from you, so I implemented it myself here #1296 Awarding you for the effort nonetheless /award 50
@Nithin-532 great job! it's such a simple change. One small request and probably that might take some fiddling around but even tho the folder state is preserved, for some reason...
> But I think this is not an optimal approach. > I nested through all the folders and children of those folders to find the path of the folderId that...
Congrats on your first contribution @Nithin-532 🎉