Skip to content

feat:News follow#51

Open
ahmedelbatal1 wants to merge 3 commits into
develop-backendfrom
feat/FollowNewsPage
Open

feat:News follow#51
ahmedelbatal1 wants to merge 3 commits into
develop-backendfrom
feat/FollowNewsPage

Conversation

@ahmedelbatal1

Copy link
Copy Markdown

Summary

Test plan

  • dotnet test backend/CCE.sln green
  • pnpm nx run-many -t lint,test green
  • If API surface changed: ./scripts/check-contracts-clean.sh green
  • If UI changed: pnpm nx run-many -t e2e (web-portal-e2e + admin-cms-e2e) green
  • Manual smoke notes (if any):

Security checklist

  • No new secrets / credentials in code
  • AuthN / AuthZ impact considered
  • Input validation on new endpoints
  • Audit-log entry for new state-changing operations

BRD traceability

Screenshots / output (optional)

}

public void ReFollow(System.Guid modifiedBy, ISystemClock clock)
{

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

why you are using refollow insted of follow

{
if (interestTopicIds is null)
throw new DomainException("interestTopicIds collection cannot be null.");
UserInterestTopics.Clear();

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

no need for this file

"Messaging": {
"Transport": "InMemory",
"UseAsyncDispatcher": true,
"UseAsyncDispatcher": false,

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

ignore the appsetting.json

using CCE.Application.Common;
using CCE.Application.Common.Interfaces;
using CCE.Application.Identity.Dtos;
using CCE.Application.InterestManagement.Dtos;

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

ignore this file

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants