Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(ui): Remove direct catalog usage #163

Merged
merged 1 commit into from
Sep 27, 2023

Conversation

lordrip
Copy link
Member

@lordrip lordrip commented Sep 27, 2023

Context

Currently, we're using the catalog at runtime by:

import catalogIndex from '@kaoto-next/camel-catalog/index.json?url';

While it works, the problem with this approach is that we're tying the @kaoto-next/camel-catalog dependency at runtime.

Changes

The change is to leverage the vite build step in which the catalog is copied to the assets folder.

    fetch(`.${DEFAULT_CATALOG_PATH}/index.json`)

Relates to: #131

@lordrip lordrip requested a review from a team September 27, 2023 14:34
Currently, we're using the catalog at runtime by:

```
import catalogIndex from '@kaoto-next/camel-catalog/index.json?url';
```

While it works, the problem with this approach is that we're tying
the @kaoto-next/camel-catalog dependency at runtime.

The change is to leverage the vite build step in which the catalog
is copied to the `assets` folder.

```
    fetch(`.${DEFAULT_CATALOG_PATH}/index.json`)
```

Relates to: KaotoIO#131
@lordrip lordrip force-pushed the chore/remove-catalog-direct-usage branch from 59bdaf9 to 5f5aba0 Compare September 27, 2023 14:40
@lordrip lordrip merged commit f29ab96 into KaotoIO:main Sep 27, 2023
2 checks passed
@lordrip lordrip deleted the chore/remove-catalog-direct-usage branch September 27, 2023 15:00
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.

1 participant