We should allow a module/feature to define and export custom customer profiles.

CommentFileSizeAuthor
#7 1192610.7-allow_export_profile_types-5.patch2.92 KBJulienD
#5 1192610.5-allow_export_profile_types-4.patch2.93 KBmrfelton
#4 1192610-allow_export_profile_types-4.patch3.82 KBpcambra
#3 1192610-allow_export_profile_types-3.patch3.8 KBpcambra

Comments

pcambra’s picture

pcambra
he/him
Primary language Spanish
commented

das-peter has some work done in http://drupal.org/node/1025580#comment-3969490 I think we could start from there.

pcambra’s picture

pcambra
he/him
Primary language Spanish
commented
Status: Active » Needs review

Ok, this kind of exports (customer profiles, line item types) are a little particular as there's "nothing" to export really as they should be defined by code, so it's just a dependency and fields issue.

Attaching a patch to provide an initial version of this, for giving it a quick test, I've published a dummy module in github with some customer profile types. https://github.com/pcambra/commerce_dummy_types

pcambra’s picture

pcambra
he/him
Primary language Spanish
commented
StatusFileSize
new 1192610-allow_export_profile_types-3.patch3.8 KB

And of course, the patch

pcambra’s picture

pcambra
he/him
Primary language Spanish
commented
StatusFileSize
new 1192610-allow_export_profile_types-4.patch3.82 KB

Some typos fixed

mrfelton’s picture

mrfelton commented
StatusFileSize
new 1192610.5-allow_export_profile_types-4.patch2.93 KB

Patch updated to apply against latest code.

bojanz’s picture

bojanz commented
Assigned: Unassigned » pcambra

Assigning to pcambra for final review / tweaks.

JulienD’s picture

JulienD commented
Status: Needs review » Needs work
StatusFileSize
new 1192610.7-allow_export_profile_types-5.patch2.92 KB

The patch works fine for me, it just need a little update to apply last changes of the module

I updated the patch.

JulienD’s picture

JulienD commented
Status: Needs work » Needs review
maciej.zgadzaj’s picture

maciej.zgadzaj commented
Status: Needs review » Reviewed & tested by the community

#7 applies cleanly against dev, seems to work fine for me.

bojanz’s picture

bojanz commented
Status: Reviewed & tested by the community » Fixed

Committed.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.