It's stable right? Any thought?

CommentFileSizeAuthor
#5 promote_remote_import-2865478-5.patch486 byteshelmo
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

helmo created an issue. See original summary.

bgm’s picture

bgm CreditAttribution: bgm at Coop SymbioTIC commented

I agree! I use it regularly.

memtkmcc’s picture

memtkmcc CreditAttribution: memtkmcc at Omega8.cc commented

We have documented Remote import in BOA context a long time ago, and it just works, I think.

colan’s picture

colan
he/him
Primary language English
Location Toronto 🇨🇦
CreditAttribution: colan at Colan Schwartz Consulting commented

Good idea. It wasn't working when I came across it, but I got it working again. (See my last two commits.)

I've been using it since without problems, even though the warnings from #2826739: Overriden Provision_Service_remote_import_hostmaster methods incompatible with parent are rather annoying. Would it be fine to ship in Experimental with these?

helmo’s picture

helmo CreditAttribution: helmo as a volunteer and at Initfour websolutions commented
Status: Active » Needs review
FileSize
promote_remote_import-2865478-5.patch486 bytes

@colan It's already in golden contrib... this just upgrades it to another section on the admin/hosting page

colan’s picture

colan
he/him
Primary language English
Location Toronto 🇨🇦
CreditAttribution: colan at Colan Schwartz Consulting commented
Status: Needs review » Reviewed & tested by the community

Right. Fine with it, unless you guys want to wait for #2826739: Overriden Provision_Service_remote_import_hostmaster methods incompatible with parent, which would keep everything green.

  • helmo committed d172868 on 7.x-3.x 
    Issue #2865478 by helmo: Promote 'Remote import' from experimental to...
helmo’s picture

helmo CreditAttribution: helmo as a volunteer and at Initfour websolutions commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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