Skip to content

Conversation

@ziyunp
Copy link
Contributor

@ziyunp ziyunp commented Dec 16, 2025

No description provided.

@ziyunp ziyunp changed the base branch from main to zpoh/add-components-lib December 16, 2025 23:34
@ziyunp ziyunp changed the title Zpoh/components readme [Components] Update README Dec 16, 2025
@ziyunp ziyunp force-pushed the zpoh/components-readme branch from 8b4585f to 7f63ffe Compare December 17, 2025 00:16
Base automatically changed from zpoh/add-components-lib to main December 17, 2025 02:50
**Prerequisites:**

This package provides reusable React components built on top of `@osdk/react` for common Ontology SDK use cases.
- React 18+
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm actually not sure if we support React 19 😅

Copy link
Contributor Author

@ziyunp ziyunp Dec 17, 2025

Choose a reason for hiding this comment

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

In package.json, we have peerDependencies set to

"react": "^17 || ^18 || ^19",


```typescript
import {/* your components */} from "@osdk/components";
## Components
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: could go either way but I don't think including this project/planning level info is helpful


```sh
npm install @osdk/react-components
npm install @osdk/react-components @osdk/react
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we make it clear the relationship between @osdk/react-components and @osdk/react?

Copy link
Contributor

@KuberSethi KuberSethi left a comment

Choose a reason for hiding this comment

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

README generally looks good - think you might need to rebase

@ziyunp ziyunp force-pushed the zpoh/components-readme branch from 7f63ffe to d682f62 Compare December 17, 2025 04:44
@ziyunp ziyunp force-pushed the zpoh/components-readme branch from 8d98790 to 1c65ec7 Compare December 17, 2025 04:59
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.

3 participants