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

add contenttype id to list picker #894

Closed
russgove opened this issue May 7, 2021 · 6 comments
Closed

add contenttype id to list picker #894

russgove opened this issue May 7, 2021 · 6 comments
Assignees
Labels
type:enhancement New feature or enhancement of existing capability
Milestone

Comments

@russgove
Copy link
Contributor

russgove commented May 7, 2021

Please check out the documentation to see if your question is already addressed there. This will help us ensure our documentation is up to date.

Category

[ x] Enhancement

[ ] Bug

[ ] Question

Version

Please specify what version of the library you are using: [latest ]
Could we add a contentTypeID to the listpicker control. The ide would be that the list picker would only let you select lists that contained the selected content type.

@ghost
Copy link

ghost commented May 7, 2021

Thank you for reporting this issue. We will be triaging your incoming issue as soon as possible.

@ghost ghost added the Needs: Triage 🔍 label May 7, 2021
@joelfmrodrigues
Copy link
Collaborator

@russgove I think this is a very useful scenario. I think the logic of the control would have to change a little bit to accommodate this, but if the check for content type is only done if the property is provided, then I assume it won't cause any negative impact when not specified.
Would you be interested in implementing this and submitting a PR?

@joelfmrodrigues joelfmrodrigues added help wanted type:enhancement New feature or enhancement of existing capability and removed Needs: Triage 🔍 labels May 7, 2021
@russgove
Copy link
Contributor Author

yup, hope toe find some time to work on it this eek

@russgove
Copy link
Contributor Author

^^week

@joelfmrodrigues
Copy link
Collaborator

@russgove that's great, I have assigned this item to you so other people are aware you may be working on it and not try to also implement it

@AJIXuMuK
Copy link
Collaborator

AJIXuMuK commented Jun 9, 2021

Thanks @russgove for both the proposal and the PR.

It has been merged and will be included in the next release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type:enhancement New feature or enhancement of existing capability
Projects
None yet
Development

No branches or pull requests

3 participants