?
avatar tonypartridge
tonypartridge
24 Nov 2017

Standard separator is |, if you do a bulk import and use , it imports with , in the url and nothing in the to address.

I suggest we issue a warning saying along the lines of:

Bulk Import failed, we did not find the | separator

I also think we should have a toggle, enable / disable on import.

Thoughts?

avatar tonypartridge tonypartridge - open - 24 Nov 2017
avatar joomla-cms-bot joomla-cms-bot - change - 24 Nov 2017
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 24 Nov 2017
avatar franz-wohlkoenig franz-wohlkoenig - change - 24 Nov 2017
Category com_plugins Feature Request
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 25 Nov 2017

+1


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/18844.

avatar franz-wohlkoenig franz-wohlkoenig - change - 25 Nov 2017
Status New Discussion
avatar brianteeman
brianteeman - comment - 27 Nov 2017

it was your pr so you know best

avatar brianteeman
brianteeman - comment - 22 Dec 2017

@tonypartridge something for you to do over the holidays ;)

avatar tonypartridge
tonypartridge - comment - 22 Dec 2017

I’ve got a few pulls to update and add this being one ?

On 22 Dec 2017, 22:50 +0000, Brian Teeman notifications@github.com, wrote:

@tonypartridge something for you to do over the holidays ;)

You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

avatar tonypartridge
tonypartridge - comment - 25 Mar 2018

Closing as we have a PR

avatar tonypartridge tonypartridge - close - 25 Mar 2018
avatar tonypartridge tonypartridge - change - 25 Mar 2018
Status Discussion Closed
Closed_Date 0000-00-00 00:00:00 2018-03-25 00:17:29
Closed_By tonypartridge

Add a Comment

Login with GitHub to post a comment