Import Trello: include members and attachments

Brought to you by: wekan

#401 Import Trello: include members and attachments

Status: closed

Owner: nobody

Labels: None

Updated: 2015-11-25

Created: 2015-11-14

Creator: Anonymous

Private: No

Originally created by: xavierpriour

Adds the ability to map exported Trello members to existing Wekan users before performing the import.
It also downloads the existing attachments and sets proper card covers.

PS: @mquandalle I have a residual UI bug, I could not manage to get a default focus() on the user easy-search field to select mapping - could you have a look? client/components/import/import.js:251

Discussion

Anonymous - 2015-11-14

Originally posted by: mquandalle

Hey Xavier, thank you very much for yet another great contribution :-) I’ll review it this week-end.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2015-11-16

Originally posted by: xavierpriour

Good news, attachment import went very quickly, so it's included as well!

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2015-11-17

Originally posted by: xavierpriour

@mquandalle all fixed.
Sorry about the muddy git history but when I rebase, I get errors when I push to github

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2015-11-25

Originally posted by: xavierpriour

ping @mquandalle 👋

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2015-11-25

Ticket changed by: mquandalle

status: open --> closed
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2015-11-25

Originally posted by: mquandalle

Hey Xavier,

Sorry for the long delay, I’m currently traveling and I’m not as productive as I would like (as you can see in my GitHub profile calendar). This is now merged, thank you again :-)

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Anonymous - 2015-11-25

Originally posted by: mquandalle

Postponed the focus issue since there is a high chance that we move away from Blaze to React (see [#299]) sooner than latter which would change the rendering engine runtime, let's keep the XXX comment as you did for now.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Log in to post a comment.