-
Notifications
You must be signed in to change notification settings - Fork 92
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
Use NcSelect
instead of NcMultiselect
in the components
#3743
Comments
|
I think it doesn't have to be a breaking change if we do it right. |
Could review a non-breaking PR if you'd be so inclined then :) |
@mejo- or @julien-nc Would one of you have time to move |
@raimund-schluessler Yes sure, done in #4247 |
Closed with #4247. |
Since
NcMultiselect
got deprecated, we should stop using it in the components itself and useNcSelect
instead.Means, we need to migrate here:
NcActionInput
UseNcSelect
forNcActionInput
type multiselect #3760NcTimezonePicker
UseNcSelect
forNcTimezonePicker
#3781NcSettingsSelectGroup
ImproveNcSettingsSelectGroup
- Convert toNcSelect
#4120https://github.com/nextcloud/nextcloud-vue/blob/4ac3029b10dc54f68a451f3496bbf6bef5272a37/src/components/NcSettingsSelectGroup/NcSettingsSelectGroup.vue#L25-L41
NcSearch
Use NcSelect in NcRichtext #4247https://github.com/nextcloud/nextcloud-vue/blob/026afe9a4c8959be853cccbef825fe7b6da8398e/src/components/NcRichText/NcReferencePicker/NcSearch.vue#L3-L10
NcProviderList
Use NcSelect in NcRichtext #4247https://github.com/nextcloud/nextcloud-vue/blob/026afe9a4c8959be853cccbef825fe7b6da8398e/src/components/NcRichText/NcReferencePicker/NcProviderList.vue#L3-L8
@Pytal @susnux Just for reference.
The text was updated successfully, but these errors were encountered: