discipletools / disciple-tools-import Goto Github PK

If the csv has an empty row in the middle, don't try to import that row.

Special characters with tildes (eg. á, é, í, ó, ú, ñ, ü) aren't being encoded correctly.

Fabián should be Fabián
Ailén should be Ailén
etc.

If you change the DT system settings to Spanish, the post_type value for 'contacts' becomes 'contactos', and results in the following error.

Failed to load resource: the server responded with a status of 400 () {"code":"rest_invalid_param","message":"Par\u00e1metro(s) no v\u00e1lido(s): post_type","data":{"status":400,"params":{"post_type":"contactos is not a valid post type"},"details":{"post_type":{"code":"rest_invalid_param","message":"contactos is not a valid post type","data":{"status":400}}}}}

Expected behavior
The post_type field should have the non-locale name: in this case, 'contacts'

Choose a mapping:

values seem mapped:

value sent to the API is off:

(separate issue)then shows up on the record as:

Export contacts with Export CSV function.
Import using this plugin. Contact with connections error out.

On the groups tab when mapping an address column
Choosing the Address field from the "standard fields" list does not seem to work,
but choosing the Address field from the "other fields" does work

I recently did an import and got an error warning that two rows weren't formatted right. I almost completely missed the warning and just clicked the green import button. I went ahead and clicked import and it imported lots of rows until it hit the error...looked like it even completed a few rows after the error, but not all the properly formatted rows.

I think it should throw a pop-up error warning if you click the green import button and it is unlikely to import properly.

A partial import seems worse than a failed import.

Also, it wasn't easy to identify the improperly formatted records, though the warning knew it was 2 out of 150... it didn't highlight or show which two it was or what/where the error was.

right now it just says:

use utf-8 file format

Have a option that can be enabled to have D.T try to geolocate address provided in the contact_address field.

Requires ability to Geolocate from the server.

After selecting your file and default source and assigned to, etc, you map the fields and then go to the verification page.

If on the verification page you notice something isn't right, or the notes tell you that there is an error (which should be highlighted in a pop-up modal or something more than the little text line above the green import button [separate issue]), when you click "Back - something's wrong" it takes you back to the complete start of mapping rather than letting you fix the mapping settings you were just verifying.

Can the back button take you back to the mapping page and retain all your settings so you can edit one and verify again rather than re-mapping 20 fields some with lots of sub-options?

Allow Tags to be set in the csv with will map to the Tags field.

Name the ID of the user to assign the contact to, or a value to be mapped to the correct user.

Uploaded CSVs currently stay publicly available in the uploads/ folder.
The csv file contains sensitive information so it should never be public and should be deleted when the process is complete.

When importing a file
( ! ) Deprecated: auto_detect_line_endings is deprecated in /wp-content/plugins/disciple-tools-import/includes/admin/class-dt-import.php on line 706

Create the ability to bulk import users from a csv.

Test on single site and multisite.
Test when some of the usernames and emails address are already present on the subsite or on a different subsite.

Upgrade utf8_decode() to mb_convert_encoding($item, "UTF-8", mb_detect_encoding($item));
see https://stackoverflow.com/questions/74865684/php-utf8-en-decode-deprecated-what-can-i-use

Add an Admin notice if the mb string extension is not installed.
see disciple_tools_import_hook_admin_notice()

Allow for connection columns in the csv so contacts can be imported and connected to groups, baptizers, etc
The csv value can be the ID(s) of the corresponding post.