ImportPagesCSV icon indicating copy to clipboard operation
ImportPagesCSV copied to clipboard

Extended list of Fieldtypes with FieldtypeOptions

Open arjenblokzijl opened this issue 10 years ago • 5 comments

Hi Ryan,

Since we use the Fieldtype Options a lot would you mind adding it?

Thanks,

Arjen

arjenblokzijl avatar Aug 06 '15 14:08 arjenblokzijl

Hi Arjen,

Did you find that it works, just by adding the FieldtypeOptions to the allowed list as shown in this PR?

Thanks, Ryan

On Thu, Aug 6, 2015 at 10:27 AM, Arjen Blokzijl [email protected] wrote:

Hi Ryan,

Since we use the Fieldtype Options a lot would you mind adding it?

Thanks,

Arjen

You can view, comment on, or merge this pull request online at:

https://github.com/ryancramerdesign/ImportPagesCSV/pull/11 Commit Summary

  • Extended list of Fieldtypes with FieldtypeOptions

File Changes

  • M ImportPagesCSV.module https://github.com/ryancramerdesign/ImportPagesCSV/pull/11/files#diff-0 (3)

Patch Links:

  • https://github.com/ryancramerdesign/ImportPagesCSV/pull/11.patch
  • https://github.com/ryancramerdesign/ImportPagesCSV/pull/11.diff

— Reply to this email directly or view it on GitHub https://github.com/ryancramerdesign/ImportPagesCSV/pull/11.

ryancramerdesign avatar Aug 09 '15 09:08 ryancramerdesign

Hey Ryan,

I imported 2000+ users this morning with besides other fields also several FieldtypeOptions fields. Works fine here.

arjenblokzijl avatar Aug 14 '15 13:08 arjenblokzijl

See https://github.com/ryancramerdesign/ImportPagesCSV/pull/13/commits/aeda8471ec26137f42d0d255d52c608b208fd526 for multi-option support.

rwetzlmayr avatar Jun 03 '16 07:06 rwetzlmayr

Thanks @rwetzlmayr! Nice additions.

arjenblokzijl avatar Jun 03 '16 08:06 arjenblokzijl

@ryancramerdesign This PR can be closed because the "Option" Fieldtype is in Version 1.0.8 included. Thanks

Ralf99 avatar Jun 03 '21 10:06 Ralf99