Problem/Motivation

As of 8.8.x, modules can declare their Drupal 8 (or 9) upgrade status (changelog, issue #2936365: Migrate UI - allow modules to declare the state of their migrations).

As of address-8.x-1.9:

Proposed resolution

Declare the status of the three migrations in a migrations/state/address.migrate_drupal.yml file.

For the unfinished migrations, add comments with a link to the issue where the migration work is ongoing.

Remaining tasks

  1. Write a patch
  2. Review and feedback
  3. RTBC and feedback
  4. Commit
  5. Release

User interface changes

None.

API changes

None.

Data model changes

None.

CommentFileSizeAuthor
#5 interdiff--4-5.txt919 bytesmparker17
#5 3189605-5--declare-upgrade-status.patch1.95 KBmparker17
#4 interdiff--2-4.txt1013 bytesmparker17
#4 3189605-4--declare-upgrade-status.patch1.95 KBmparker17
#2 3189605-2--declare-upgrade-status.patch842 bytesmparker17

Comments

mparker17 created an issue. See original summary.

mparker17’s picture

mparker17
he/him
Primary language English
Location UTC-5
commented
Issue summary: View changes
Status: Active » Needs review
StatusFileSize
new 3189605-2--declare-upgrade-status.patch842 bytes

Here's a patch. Reviews welcome.

mparker17’s picture

mparker17
he/him
Primary language English
Location UTC-5
commented
Issue summary: View changes
Related issues: +#3173870: Support core migrations for automated update from Location (D7) fields, +#2974631: Add D7 Location-to-Address migrate process plugin, +#2912087: Field plugin migration from D7/addressfield

(added related issues)

mparker17’s picture

mparker17
he/him
Primary language English
Location UTC-5
commented
StatusFileSize
new 3189605-4--declare-upgrade-status.patch1.95 KB
new interdiff--2-4.txt1013 bytes

Here's a patch that adds a test for the update status.

mparker17’s picture

mparker17
he/him
Primary language English
Location UTC-5
commented
StatusFileSize
new 3189605-5--declare-upgrade-status.patch1.95 KB
new interdiff--4-5.txt919 bytes

Fix test breakage and coding standards issues.

  • bojanz committed 8b301b9 on 8.x-1.x authored by mparker17 
    Issue #3189605 by mparker17: Declare upgrade status from D6, D7
bojanz’s picture

bojanz commented
Status: Needs review » Fixed

Committed, thanks!

mparker17’s picture

mparker17
he/him
Primary language English
Location UTC-5
commented

Awesome! Thank you!

mparker17’s picture

mparker17
he/him
Primary language English
Location UTC-5
commented

I've just updated the patches in #2974631: Add D7 Location-to-Address migrate process plugin and #3173870: Support core migrations for automated update from Location (D7) fields, which declare their migrate statuses.

Status: Fixed » Closed (fixed)

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