Read the standard for parameter aliasing here - https://github.com/hassanhabib/CSharpCodingStandard/blob/master/Methods.md#104-passing-parameters

Target file: https://github.com/hassanhabib/Taarafo.Core/blob/master/Taarafo.Core/Models/Posts/Exceptions/PostDependencyException.cs

The base method takes a parameter name 'message' which is not being aliased here.

@hassanhabib please assign to me

@hassanhabib @ShriHumrudha should this use the QueryTrackingBehavior.NoTracking behaviour?

broker.ChangeTracker.QueryTrackingBehavior = QueryTrackingBehavior.NoTracking;

(and possibly the same for SelectAllPosts?)

@hassanhabib please assign to me

We should always keep our code neat & tidy.

Targeting file: https://github.com/hassanhabib/Taarafo.Core/blob/master/Taarafo.Core/Brokers/Loggings/LoggingBroker.cs

1. Remove one extra line between line number 4 & 7
2. Order the implementation method to match the order in the interface

Addition of Author so we can identify who submitted the comment.

Target file: https://github.com/hassanhabib/Taarafo.Core/blob/master/Taarafo.Core/Startup.cs

Reference file:https://github.com/hassanhabib/OtripleS/blob/master/OtripleS.Web.Api/Startup.cs

Refactor scope: The service registration for the brokers can be grouped under one method 'AddBrokers'

Target File Line 22: https://github.com/hassanhabib/Taarafo.Core/blob/master/Taarafo.Core.Tests.Unit/Services/Foundations/Posts/PostServiceTests.Logic.RetrieveAll.cs

Reference: https://github.com/hassanhabib/The-Standard/blob/master/2.%20Services/2.1%20Foundations/2.1%20Foundations.md#2130-abstraction

The Standard: The expected entity should be a standalone reference from the storage entity

@hassanhabib please assign to me

Learn more about acceptance tests here:
https://www.youtube.com/watch?v=WWN-9ahbdIU

Learn more about acceptance tests here:
https://www.youtube.com/watch?v=WWN-9ahbdIU

This will ensure that code cleanup will place system using directives at the top
This will add the file header template / copyright text to the top of files during code cleanup

These settings are applied via:

#Formatting - organize using options

#sort System.* using directives alphabetically, and place them before other usings
dotnet_sort_system_directives_first = true

#Formatting - header template

#add file header template
file_header_template = ---------------------------------------------------------------\nCopyright (c) Coalition of the Good-Hearted Engineers\nFREE TO USE TO CONNECT THE WORLD\n---------------------------------------------------------------

Read the standard for parameter aliasing here - https://github.com/hassanhabib/CSharpCodingStandard/blob/master/Methods.md#104-passing-parameters

Target file: https://github.com/hassanhabib/Taarafo.Core/blob/master/Taarafo.Core/Models/Posts/Exceptions/PostServiceException.cs

The base method takes a parameter name 'message' which is not being aliased here.

Hello Eng/ Hassan

Here's my suggestion for Comment model

RelationShip for the model I think about

• Post will have one or many comments,
• Author will have one or many comments,
• Comment itself will have no comment or one , many comments ( i.e replies on a comment)

i'm not sure about comment state will be suitable or instead add comment rate
some kind of enum contains rates or status of comment

Validation check currently checks that updated date is the sames as the updated date from storage. We expect the date to change on an update.