Skip to content

chore(dashboards-demo): add rootDir to resolve TypeScript path mapping errors - #2319

Open
chintankavathia wants to merge 1 commit into
mainfrom
chore/dashboards/tsconfig-path-errors
Open

chore(dashboards-demo): add rootDir to resolve TypeScript path mapping errors#2319
chintankavathia wants to merge 1 commit into
mainfrom
chore/dashboards/tsconfig-path-errors

Conversation

@chintankavathia

@chintankavathia chintankavathia commented Jul 13, 2026

Copy link
Copy Markdown
Member

Set rootDir to '../..' in tsconfig.app.json to allow imports from @siemens/element-ng/* and other library packages via path mappings without triggering 'not under rootDir' TypeScript errors.


Documentation.
Examples.
Dashboards Demo.
Playwright report.

Coverage Reports:

Code Coverage

…g errors

Set rootDir to '../..' in tsconfig.app.json to allow imports from
@siemens/element-ng/* and other library packages via path mappings
without triggering 'not under rootDir' TypeScript errors.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request updates the TypeScript configuration for the dashboards-demo application (tsconfig.app.json) by adding the rootDir property set to "../.." under compilerOptions. There are no review comments, and I have no additional feedback to provide.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

@chintankavathia
chintankavathia marked this pull request as ready for review July 13, 2026 10:57
@chintankavathia
chintankavathia requested review from a team as code owners July 13, 2026 10:57
@spike-rabbit

Copy link
Copy Markdown
Member

According to https://devblogs.microsoft.com/typescript/announcing-typescript-7-0/ can also just set an input instead.

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.

2 participants