Skip to content

feat: whitelist creation,revision,citation,license in collections pro… - #337

Merged
utas-raymondng merged 1 commit into
mainfrom
feature/9001-whitelist-seo-properties
Aug 27, 2026
Merged

feat: whitelist creation,revision,citation,license in collections pro…#337
utas-raymondng merged 1 commit into
mainfrom
feature/9001-whitelist-seo-properties

Conversation

@utas-amberbian

@utas-amberbianutas-amberbian commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Whitelist creation, revision, citation, and license in the collections properties filter. The portal SEO pipeline needs them for Dataset JSON-LD (dates, license, citation), but the filter currently rejects them.

@utas-raymondngutas-raymondng left a comment

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.

Do not think there is a need to make changes here as we already have those value in the UI, any reason why we cannot use existing function but new one?

@utas-amberbian

Copy link
Copy Markdown
ContributorAuthor

Do not think there is a need to make changes here as we already have those value in the UI, any reason why we cannot use existing function but new one?

The UI gets these values from the single-record endpoint (/collections/{id}), which returns the full document — no properties filter involved. That's why the detail page can already show them.

The SEO pipeline can't use that endpoint: it builds pages for all ~16k records weekly, so it uses the bulk endpoint with ?properties=... to avoid 16k individual requests.

@utas-raymondngutas-raymondng left a comment

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.

LGTM

@utas-raymondng
utas-raymondng merged commit f9e55b9 into mainAug 27, 2026
7 checks passed
@utas-raymondng
utas-raymondng deleted the feature/9001-whitelist-seo-properties branch August 27, 2026 06:11
Sign up for freeto 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

@utas-amberbian@utas-raymondng