ContentItemsBlock

Description

None

Technical tasks

None

Deployment actions

None

Activity

Show:
Marijn Otte
January 20, 2016, 9:27 AM

I have some things to discuss. The main thing is why you created a separate formType, instead of adding an option to the current one. I think it is better to adjust the current one to support Ajax.

Johan Liefers
January 20, 2016, 9:36 AM

That is because they both work in a different way. The original select2 is based on the choice formtype and expects options (as array or just string) to be predefined, and just changes the appearance of the selectbox. The ajax component is based on database content, and returns content (or an array of content) instead of a string or array.

Just like there is a different formtype for a document, I think this should also be another formtype.

Marijn Otte
January 21, 2016, 12:28 PM

Please rename to integrated_content_choice.

Thanks for the documentation!

Johan Liefers
January 21, 2016, 1:21 PM

done! the documentation does still have the old name, can you edit that yourself?

Marijn Otte
January 21, 2016, 1:28 PM

that's fine. Please set the issue to Done when you are ready

Assignee

Unassigned

Client

None

Fix versions

Configure