Skip to content
This repository has been archived by the owner on Apr 11, 2023. It is now read-only.

Suggestion: stop dealing with the model #44

Open
chrisfls opened this issue Jun 16, 2021 · 0 comments
Open

Suggestion: stop dealing with the model #44

chrisfls opened this issue Jun 16, 2021 · 0 comments

Comments

@chrisfls
Copy link

I disagree with how this library deals with the user's model:

My suggestion is to change them to plain items : List item and selection : Selection item.

I can't see any advantages in keeping those *FromModel functions, and they also add a cumbersome model type var everywhere.

... Of course, we could just use always to skip the model requirement, and this is what I've done when integrating this library with paack-ui. But the current API doesn't seem either ideal or standard.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant