Hi.

This patch adds support for importing csv data using drush rather than configuration UI.
The code ain't perfect, but works (copies import form's code rather than calling it).

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

hkoosha created an issue. See original summary.

hkoosha’s picture

hkoosha’s picture

The patch will fail because the module will be in .../weather-weather rather than .../weather (composer namespace issues thing).
So I manually ran the tests, it seems ok. (But the tests has no use anyway, they won't test drush).

akyrylenko’s picture

Patch reviewed and works well.
Version: 8.x-1.x-dev

Updated patch applying coding standards added.

ysamoylenko’s picture

Status: Needs review » Reviewed & tested by the community

Hi @AK47,

Thanks for your efforts.

The patch will be committed soon.

  • ysamoylenko committed 7105941 on 8.x-1.x authored by AK47
    Issue #3086718 by hkoosha, AK47, ysamoylenko: Add drush command to...
ysamoylenko’s picture

Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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