backdrop-contrib / d2b_migrate

GNU General Public License v2.0
5 stars 6 forks source link

Some suggestions from my experience with this module #47

Open AlexHoebart-ICPDR opened 3 months ago

AlexHoebart-ICPDR commented 3 months ago

This looks like a great work and could be very useful. Thanks! Some feedback for potential improvements based on my experience when trying to use this for migration of one of my sites:

  1. In Database Analysis, some modules are reported "In core (Disabled in source)" - but they are enabled in source in my case: admin_menu, ctools, file.
  2. It would be useful to see the module machine name in the list.
  3. AutoUpload is reported as "Not yet ported", but as far as I understand the functionality (initiates automatic upload of files and removes the Upload button) is in Backdrop core.
  4. Bootstrap is reported as Type Module - it is a Theme.
  5. D7Security client is reported as "Not yet ported" - I think that specific module could be ignored.
  6. Some module downloads fail (which is not the fault of this module). After installing them manually and coming back to this module's page, the Database Analysis needs to be started again (which might take a while on site with a lot of modules) - it would be nice if that list is cached.
  7. The Database Analysis reports modules as "Already downloaded" - it would even more useful if it also would report "Already enabled" if that is the case.

In the end, the last step of the database update did not succeed for me as it only reports "No pending updates.". I see Drupal database imported, but Backdrop does not upgrade it and accessing the home page fails with "Unknown column 'base.type' in .... {file_managed}". But I assume this problem is beyond the scope of this module.