Skip to content

Fix for crash in navbar with double ambient modules#47395

Merged
DanielRosenwasser merged 2 commits intomainfrom
navBarDoubleAmbientModule
Jan 12, 2022
Merged

Fix for crash in navbar with double ambient modules#47395
DanielRosenwasser merged 2 commits intomainfrom
navBarDoubleAmbientModule

Conversation

@DanielRosenwasser
Copy link
Copy Markdown
Member

Fixes #47394.
Fixes #38058.

@DanielRosenwasser DanielRosenwasser merged commit 14f33d5 into main Jan 12, 2022
@DanielRosenwasser DanielRosenwasser deleted the navBarDoubleAmbientModule branch January 12, 2022 02:14
@DanielRosenwasser
Copy link
Copy Markdown
Member Author

@typescript-bot cherry-pick this to release-4.5

@typescript-bot
Copy link
Copy Markdown
Collaborator

typescript-bot commented Jan 13, 2022

Heya @DanielRosenwasser, I've started to run the task to cherry-pick this into release-4.5 on this PR at 3ba8469. You can monitor the build here.

@typescript-bot
Copy link
Copy Markdown
Collaborator

Hey @DanielRosenwasser, I've opened #47421 for you.

typescript-bot pushed a commit to typescript-bot/TypeScript that referenced this pull request Jan 13, 2022
Component commits:
f230a5a Add failing test.

3ba8469 Guard against undefined module bodies in navbar/navtree.
DanielRosenwasser added a commit that referenced this pull request Jan 13, 2022
Component commits:
f230a5a Add failing test.

3ba8469 Guard against undefined module bodies in navbar/navtree.

Co-authored-by: Daniel Rosenwasser <[email protected]>
@microsoft microsoft locked as resolved and limited conversation to collaborators Oct 22, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Author: Team For Milestone Bug PRs that fix a bug with a specific milestone

Projects

None yet

Development

Successfully merging this pull request may close these issues.

navbar crash from duplicate body-less ambient modules TypeError: Cannot read property 'kind' of undefined in 'areSameModule' during 'navtree'

3 participants