Skip to content

fix(): dep mismatch - #774

Merged
agviegas merged 1 commit into
ThatOpen:mainfrom
ShaMan123:fix/deps
Jul 24, 2026
Merged

fix(): dep mismatch#774
agviegas merged 1 commit into
ThatOpen:mainfrom
ShaMan123:fix/deps

Conversation

@ShaMan123

@ShaMan123ShaMan123 commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Description

Fixes raycaster bug due to version mismtach (getLocalIdsFromItemIds not on frags v3.4.3)

related to #737#747#773 - not confirmed as the sole fix

Additional context


What is the purpose of this pull request?

  • Bug fix
  • New Feature
  • Documentation update
  • Other

Before submitting the PR, please make sure you do the following:

  • Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
  • Follow the Conventional Commits v1.0.0 standard for PR naming (e.g. feat(examples): add hello-world example).
  • Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
  • Ideally, include relevant tests that fail without this PR but pass with it.

@ShaMan123ShaMan123 left a comment

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

ready

const fragments = this.components.get(FragmentsManager);
const model = fragments.list.get(result.modelId);
if (!model) return null;
const localIds = await (model as any).getLocalIdsFromItemIds([

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

was masking the bug - lint would have caught this

@agviegas
agviegas merged commit 2be954d into ThatOpen:mainJul 24, 2026
@ShaMan123
ShaMan123 deleted the fix/deps branch July 24, 2026 12:34
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

@ShaMan123@agviegas