Skip to content
This repository was archived by the owner on May 19, 2024. It is now read-only.

#108 Blog owner can now enable or disable comments for blog post - #148

Open
manojkulkarni30 wants to merge 1 commit into
VenusInterns:masterfrom
manojkulkarni30:108
Open

#108 Blog owner can now enable or disable comments for blog post#148
manojkulkarni30 wants to merge 1 commit into
VenusInterns:masterfrom
manojkulkarni30:108

Conversation

@manojkulkarni30

Copy link
Copy Markdown

Implemented the functionality to enable or disable comments for blog post.

  • If comments are disabled for the blog post, then a message will be displayed on blog post detail page that Comments are disabled for this post.
  • If there were some existing public comments for the blog post before disabling the comments, then those comments will be displayed along with the message.

@julietdaniel , @lucasisaza @umaslakshmi please review the pull request. Let me know if you have any suggestions.

rootNode.Add(new XElement("LastModified", post.LastModified.ToString("o")));
rootNode.Add(new XElement("IsPublic", post.IsPublic.ToString()));
rootNode.Add(new XElement("Excerpt", post.Excerpt));
rootNode.Add(new XElement("EnableComments", post.EnableComments));

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you have to convert this to a string first? Like post.EnableComments.ToString()? That is what we were doing for IsPublic, but maybe it's not necessary.

@lucasisaza lucasisaza left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, thanks for helping us out!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants