Please add comments section about blogifier HOT 9 OPEN

Hi,
I tried to add the comment function myself.

• People and post comment and edit after posting:

• The comments have a simple tree structure to collect the sub-comments which responds another comment. User can choose to fold or unfold it:

• Comment needs user to login to post comment or thumb-up/revoke thumb-up. Post author had a 'Author' tab in pill shape:
  

I haven't raise the PR yet, because I haven't communicated with the repo owner about the solution, yet.

from blogifier.

Solution:

• New table 'Comments' to store comments and user id who posted them, foreign key to 'Posts' table;
• New table 'CommentsLikes' to store thumb-ups and user id, foreign key to 'Comments' table;
• Embed a Blazor Server to 'Blogifier' project, hosting and embeding 'Comments.razor' into each post.cshtml;
• 'CommentCard.razor' show the contents, and 'CommentEditor.razor' show the easyMDE editor to edit or post new comment, are also embeded into 'Comments.razor';

Flaws:

• Blazor Server embeded into existing MVC can not share httpcontext, so the Login function may need changes to avoid user login again for comment;
• As for myself, I changed the Auth funciton to SSO using IdentityServer4, so I haven't study how the original Blogifier project should handle it.

Full size screenshot:

from blogifier.

@LinQiaoPorco
hello
Whether you submit your implemented comment feature to a pr
I'm looking forward to implementing comments
This project is being upgraded, please push it to the dev branch if possible

from blogifier.

Hi, I haven't looked into the project for quite a long time. I will try to PR from latest branch to see if it works.
My previous repo was revised to use OpenID connect, so it is not proper to PR the original one.

I found that dev branch is in .NET 7 already, while I did not plan to hasve .NET 7 SDK. Can I just pr to main branch?

from blogifier.

It can be in the mian-bak branch, the mian branch was wrongly submitted by me and I am asking the warehouse manager to fix it, [mian-bak] is the original warehouse code
But the .net7 update in the dev branch will be updated to the mian branch soon. If you submit to the mian branch, the request will probably not be merged.

from blogifier.

You can try to push to the dev branch and we can solve the problem together

from blogifier.

OK. That will be a messy though. BTW, did you read description on my solution? Would that be OK to progress?

from blogifier.

Yes, there may be a big difference in the login plan
My idea is to use the code that can be used as much as possible to realize the comment function, structure table, logic, etc... even css..

from blogifier.

I remember that the changes target comments function added since this commit.

from blogifier.