Skip to content

Conversation

igdmdimitrov
Copy link

No description provided.

@igdmdimitrov igdmdimitrov added the status: do-not-merge PR is not ready/risky for merging label Oct 8, 2025
@igdmdimitrov igdmdimitrov marked this pull request as ready for review October 14, 2025 07:48
@igdmdimitrov igdmdimitrov changed the title chore(*): update to igniteui-react 19.3.0-rc.2 feat(samples): migrate to vite Oct 14, 2025
@dkamburov dkamburov requested a review from skrustev October 14, 2025 08:12
@igdmdimitrov igdmdimitrov added awaiting-test and removed status: do-not-merge PR is not ready/risky for merging labels Oct 14, 2025
@gedinakova gedinakova self-assigned this Oct 14, 2025
import { defineConfig } from 'vite';
import react from '@vitejs/plugin-react';

export default defineConfig({
Copy link
Member

Choose a reason for hiding this comment

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

The samples browser should be able to run both licensed and unlicensed Ignite UI packages. As it is currently configured it will not run correctly if you add @infragistics/ in front of the packages. They are described in the tsconfig.paths.json.
I was able to correctly run vite using the vite-tsconfig-paths plugin, so it can read the tsconfig.

Also I remember there was some kind of issue related to the process.env being undefined, but this might have been fixed and not be releavant.

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.

3 participants