Download & Extend
Migrate » Issues

Display Migration's group on UI

Posted by Haza on November 23, 2011 at 10:17am
Project:Migrate
Version:7.x-2.x-dev
Component:migrate_ui
Category:feature request
Priority:normal
Assigned:Unassigned
Status:closed (fixed)
Issue tags:Migrate 2.6

Issue Summary

Hi,

It would be great to display the group name of a migration on the migration dashboard (and when we use the "drush ms" command) to be able to quickly see and a migration use the group feature and what migration belong to wich group.

Login or register to post comments

Comments

#1

Posted by Haza on November 24, 2011 at 3:04pm
Status:active» needs review

The patch is attached.

Group is display on "admin/content/migrate" and also when using "drush ms"

A quick look on what it may look like :

migrate ui

drush ms

What do you think about that ?

AttachmentSizeStatusTest resultOperations
1350284.display_migration_group.patch2.02 KBIgnored: Check issue status.NoneNone

#2

Posted by mikeryan on November 26, 2011 at 6:59pm
Status:needs review» needs work

Actually, what I always had in mind (but never got around to implementing) is visually grouping the migrations, something like this for drush ms:

Group: default         Total  Imported  Unimported  Status     Last imported      
BeerTerm              3      3         0           Idle       2011-11-26 11:54:04
BeerUser              4      4         0           Idle       2011-11-26 11:54:05
BeerNode              3      3         0           Idle       2011-11-21 17:38:18
BeerComment           5      0         5           Idle       2011-11-21 17:38:36

Group: wine            Total  Imported  Unimported  Status     Last imported      
WinePrep              N/A    N/A       N/A         Idle                          
WineBestWith          3      0         3           Idle       2011-11-21 17:38:15

#3

Posted by drewish on November 28, 2011 at 6:42pm

+1 on the grouping. i'd rather not see the list get any wider.

#4

Posted by marvil07 on April 13, 2012 at 2:32am
Status:needs work» needs review

Here a patch for migrate-status output.

I first tried with one drush table per group, but I end up doing it all in one table because IMHO that looks easier to read(all columns at the same width).

In the other side, I guess it is ok to have one more column on the web version at migrate_ui. We could have the grouped html tables(but yet it could be the same problem I mentioned on the last paragraph).

AttachmentSizeStatusTest resultOperations
0001-Issue-1350284-Let-drush-migrate-status-show-migratio.patch4.53 KBIgnored: Check issue status.NoneNone

#5

Posted by marvil07 on April 13, 2012 at 2:35am

btw, here a ignore-all-space-version of the patch for easier review.

AttachmentSizeStatusTest resultOperations
without-space-changes.diff.txt1.66 KBIgnored: Check issue status.NoneNone

#6

Posted by wusel on April 13, 2012 at 10:40am

+1 on the grouping #5.

My suggestion for the "Migrate UI":
Add the group on all tabs "Overwiew" of the group, if needed.

#7

Posted by mikeryan on April 20, 2012 at 8:35pm
Status:needs review» active

Committed to D6 and D7, thanks!

Leaving this issue open for the sake of migrate_ui...

#8

Posted by mikeryan on November 7, 2012 at 2:41pm

Tagging for 2.6.

#9

Posted by mikeryan on November 7, 2012 at 7:22pm
Status:active» fixed

This was committed to the wizard_api branch, which is now merged to 7.x-2.x.

#10

Posted by System Message on November 21, 2012 at 7:31pm
Status:fixed» closed (fixed)

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

nobody click here