Follow-up to #2562849: [meta] Confirmation page

Right now, for each enabled module on the source site, if we have no migrations tied to importing data from the module, we simply display "No upgrade path available". We should do our best to identify the possible scenarios, giving more specific details for each one and applying "warning" or "error" highlighting as appropriate. Possible scenarios:

  1. Module with no data (tables or variables) to migrate - should not even be shown. The problem here is there is no way to programmatically determine whether the source module has data to migrate. We can hard-code this knowledge for core modules, and perhaps the most popular contrib modules, but we can't reliably determine this in the general case, short of populating module metadata on d.o and retrieving it from there. See #2569787: Handle modules with no data to migrate
  2. Module exists in the D8 installation but is not enabled. To determine this we'll need a template discovery service that examines all modules, installed or not. Provide a message "This module must be enabled for the legacy content to be upgraded" or somesuch. Is this a warning or error? See #2568525: Identify potential (but unenabled) upgrade paths
  3. Module exists in the D8 installation, is enabled, but does not provide an upgrade path (yet). What do we tell the upgrader in this case? "Please nag the module maintainer to provide an upgrade path"? Warning, perhaps, since there isn't much the upgrader can do about it. See #2569789: Handle modules which should have an upgrade path but don't
  4. Module is not present in the D8 installation. Provide a message "To upgrade this module you must install and enable the Drupal 8 version of the module". Error? See #2569793: Handle modules which aren't present

In addition, perhaps we don't allow proceeding unless the upgrader has clicked a checkbox next to each no-upgrade module, acknowledging that it's OK to proceed without them?

Sample appearance:

CommentFileSizeAuthor
#4 upgrade_analysis_report.png61.67 KBmikeryan
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

mikeryan created an issue. See original summary.

mikeryan’s picture

Issue summary: View changes
mikeryan’s picture

Issue summary: View changes
mikeryan’s picture

Issue summary: View changes
FileSize
61.67 KB

yoroy wireframe added to IS.

yoroy’s picture

In IRC chat with webchick, mikeryan and moi the latest version of the plan was summarized as:

1) Forget about advanced debugging
2) Forget about warning vs. error
3) If a source is missing a destination, it's an error.
4) You can re-run the upgrade UI as many times as you wish

So this issue is less important for now. #2568951: Option to incrementally upgrade is where it's at :-)

mikeryan’s picture

Title: [meta] More detail on source modules with no upgrade paths » [meta] Highlighting of source modules with no upgrade paths

I've postponed the child issues dealing with specific scenarios - for now, this issue is just about apply error highlighting to source modules without upgrade paths.

mikeryan’s picture

Issue tags: +Barcelona2015, +Novice

Tagging for Barcelona sprinting.

mikeryan’s picture

Status: Active » Closed (duplicate)
Issue tags: -Barcelona2015, -Novice
Related issues: +#2678638: [META] Usability refinements for Migrate UI

Any further UX refinement should go under #2678638: [META] Usability refinements for Migrate UI.