Skip to content

Chore: import algolia indexer#455

Merged
zigzagdev merged 5 commits intochore/add-main_iamge_urlfrom
chore/import-algolia-indexer
May 5, 2026
Merged

Chore: import algolia indexer#455
zigzagdev merged 5 commits intochore/add-main_iamge_urlfrom
chore/import-algolia-indexer

Conversation

@zigzagdev
Copy link
Copy Markdown
Owner

No description provided.

zigzagdev and others added 5 commits May 5, 2026 14:00
…eritagePayload

The list-path payload (used by getAllHeritages and the Algolia search
result reshape) didn't pass main_image_url, so the SummaryFactory could
never populate WorldHeritageDto::mainImageUrl and thumbnail_url stayed
null in the API. Forwards $heritage->main_image_url so the new fallback
chain in WorldHeritageViewModel resolves correctly.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…ById factory payload

The detail-path WorldHeritageDetailFactory::build call was missing the
'main_image_url' key, so the DTO's mainImageUrl stayed null and the
detail endpoint's thumbnail_url did not reflect the new field.
Forwards $heritage->main_image_url so the ViewModel fallback resolves.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Adds world_heritage_sites.main_image_url to the chunked SELECT so the
Algolia importer can read the new column off each model.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…ects

Each Algolia object now carries the raw main_image_url alongside the
existing thumbnail_url so search consumers can render via the new
field directly.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…ritages

Switches the Algolia object's thumbnail_url to main_image_url first,
falling back to images[0]->url, so the search index mirrors the new
fallback semantics now used by WorldHeritageViewModel and
WorldHeritageDtoCollection::toSummaryArray.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@zigzagdev zigzagdev changed the base branch from main to chore/add-main_iamge_url May 5, 2026 05:53
@zigzagdev zigzagdev self-assigned this May 5, 2026
@zigzagdev zigzagdev linked an issue May 5, 2026 that may be closed by this pull request
3 tasks
Copy link
Copy Markdown
Owner Author

@zigzagdev zigzagdev left a comment

Choose a reason for hiding this comment

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

Ok

@zigzagdev zigzagdev merged commit 75834dc into chore/add-main_iamge_url May 5, 2026
25 checks passed
@zigzagdev zigzagdev deleted the chore/import-algolia-indexer branch May 5, 2026 05:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: push main_image_url through Algolia indexer

1 participant