Skip to content

feat: add default author for private sources#51

Merged
MaGOs92 merged 3 commits into
masterfrom
gfay_feat_add-default-author-for-private-sources
Jun 10, 2026
Merged

feat: add default author for private sources#51
MaGOs92 merged 3 commits into
masterfrom
gfay_feat_add-default-author-for-private-sources

Conversation

@MaGOs92

@MaGOs92 MaGOs92 commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI 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.

Pull request overview

Adds support for configuring a per-source “default author” (for PRIVATE sources) and uses it as a fallback email recipient when a report is processed/ignored.

Changes:

  • Add defaultAuthor (jsonb) to Source + DB migration.
  • Add PUT /sources/:idSource endpoint with source-token ownership checks to update nom and/or defaultAuthor.
  • Update report notification logic to send emails to author.email or source.defaultAuthor.email.

Reviewed changes

Copilot reviewed 12 out of 12 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
src/modules/source/source.entity.ts Adds defaultAuthor column to Source entity and sets it on PRIVATE creation.
src/modules/source/source.dto.ts Adds DTO validation for defaultAuthor and introduces UpdateSourceDTO.
src/modules/source/source.service.ts Updates token lookup selection + adds new updateOne(id, dto) method.
src/modules/source/source.controller.ts Adds PUT /sources/:idSource endpoint.
src/modules/source/source.module.ts Registers SourceMiddleware for PUT source route.
src/modules/source/source.middleware.ts Adds PUT-specific ownership enforcement (idSource param).
src/common/base-report.service.ts Adds fallback to defaultAuthor.email for notification emails.
src/migrations/index.ts Registers new migration.
src/migrations/1781234567890-add-source-default-author.ts Adds defaultAuthor jsonb column to sources table.
src/tests/source.spec.ts Extends source tests for defaultAuthor + new PUT endpoint coverage.
src/tests/alert.spec.ts Adjusts alert serialization expectations to ignore defaultAuthor on nested source.
src/tests/signalement.spec.ts Adjusts signalement serialization expectations to ignore defaultAuthor on nested source.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/common/base-report.service.ts Outdated
Comment thread src/modules/source/source.service.ts
Comment thread src/common/base-report.service.ts
@MaGOs92 MaGOs92 merged commit d3da404 into master Jun 10, 2026
4 checks passed
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.

2 participants