Skip to content

Conversation

@joaquincasal
Copy link
Collaborator

JuliRossi and others added 6 commits September 25, 2025 16:31
* Scaffolding

* chore(closest-preview): update tsconfig.json to enable esModuleInterop and add vitest types
* adding config screen for Closest Preview

* changing interface for type

* updating fomra version + initial version of content type preview exclusion

* encapsulating live preview content types fetch
* Add sidebar ui

* Fix config screen tests

* Update test name
* removing filters and doing it through query

* generic refactors

* adding tests

* adding depth check in sidebar

* removing unused interface

* creating helpers + showing display field + modifying max depth ui

* removing depth logic

* doing filtering al the last moment

* refactor backlinks logic

* fixing tests to match the changes

* removing unused import

* renames and inline
@joaquincasal joaquincasal requested a review from a team as a code owner October 15, 2025 16:39
@ryunsong-contentful
Copy link
Contributor

ryunsong-contentful commented Oct 16, 2025

Maybe I have a slight misunderstanding how this app works

So I have this setup
Dang -> maw -> WHAT
A title -> WHAT

So then when I go to the WHAT entry, I shows both A title and Dang as the closest preview which makes somewhat sense since they're both Live Preview enabled. However, I expected A title to show up as it's "closer" in terms of node distance.

Feel free to check out the dev space as shown in the screenshot

Screenshot 2025-10-16 at 11 59 24 AM

Copy link
Contributor

@ryunsong-contentful ryunsong-contentful left a comment

Choose a reason for hiding this comment

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

Code looks good. Just double check my comment

@joaquincasal joaquincasal merged commit 9acbb90 into master Oct 16, 2025
15 checks passed
@joaquincasal joaquincasal deleted the closest-preview-app branch October 16, 2025 18:25
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.

5 participants