Skip to content

Extend "get collection" use case to return theme #417

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
vera wants to merge 6 commits into
base: develop
Choose a base branch
from
Open

Extend "get collection" use case to return theme #417

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
3bd4150
feat: return theme when reading a collection
vera Dec 10, 2025
7154910
fix: mark all collection theme attributes as optional
vera May 7, 2026
a15cfde
test: add unit test for getting collection with a theme
vera May 7, 2026
abcc7e7
docs: update CHANGELOG.md
vera May 7, 2026
cdd9ba7
test: add minimal integration test for getting collection with a theme
vera May 7, 2026
979772e
Merge remote-tracking branch 'origin/develop' into feat/get-collectio…
vera May 7, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
12 changes: 12 additions & 0 deletions src/collections/domain/models/Collection.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,18 @@ export interface Collection {
isMetadataBlockRoot: boolean
isFacetRoot: boolean
childCount: number
theme?: CollectionTheme
}

export interface CollectionTheme {
id: number
logo: string
Copy link
Copy Markdown
Contributor

@ChengShi-1 ChengShi-1 Apr 3, 2026 •
edited
Loading

Choose a reason for hiding this comment

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

After reviewing the backend, it appears that they are all optional.

tagline: string
linkUrl: string
linkColor: string
textColor: string
backgroundColor: string
logoBackgroundColor?: string
}

export interface CollectionInputLevel {
Expand Down
12 changes: 12 additions & 0 deletions src/collections/infra/repositories/transformers/CollectionPayload.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,18 @@ export interface CollectionPayload {
isMetadataBlockRoot: boolean
isFacetRoot: boolean
childCount: number
theme?: CollectionThemePayload
}

export interface CollectionThemePayload {
id: number
logo: string
tagline: string
linkUrl: string
linkColor: string
textColor: string
backgroundColor: string
logoBackgroundColor?: string
}

export interface CollectionInputLevelPayload {
Expand Down
5 changes: 4 additions & 1 deletion src/collections/infra/repositories/transformers/collectionTransformers.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ import { AxiosResponse } from 'axios'
import {
CollectionContactPayload,
CollectionInputLevelPayload,
CollectionPayload
CollectionPayload,
} from './CollectionPayload'
import { transformPayloadToOwnerNode } from '../../../../core/infra/repositories/transformers/dvObjectOwnerNodeTransformer'
import { CollectionFacet } from '../../../domain/models/CollectionFacet'
Expand Down Expand Up @@ -74,6 +74,9 @@ const transformPayloadToCollection = (collectionPayload: CollectionPayload): Col
isFacetRoot: collectionPayload.isFacetRoot,
description: collectionPayload.description,
childCount: collectionPayload.childCount,
...(collectionPayload.theme && {
theme: collectionPayload.theme
}),
...(collectionPayload.isPartOf && {
isPartOf: transformPayloadToOwnerNode(collectionPayload.isPartOf)
}),
Expand Down
Loading