Problem/Motivation

See patch

Steps to reproduce

See patch

Proposed resolution

See patch

Remaining tasks

Review and approve and commit code changes included with patch

User interface changes

Expect fresh install of WxT 6.1.x to talk about Drupal 11 not Drupal 9.

API changes

Minor

Data model changes

example content.

CommentFileSizeAuthor
#13 wxt-6.1.5.png163.41 KBsmulvih2
#10 drupal9_in_wxt62x.png11.03 KBjoseph.olstad
#8 wxt_install.png19.14 KBjoseph.olstad
#2 wxt-drupal9_to_drupal11-3562743-02.patch15.81 KBjoseph.olstad

Comments

joseph.olstad created an issue. See original summary.

joseph.olstad’s picture

joseph.olstad
Primary language French
commented
Assigned: joseph.olstad » smulvih2
Status: Active » Needs review
StatusFileSize
new wxt-drupal9_to_drupal11-3562743-02.patch15.81 KB

Review patch

  • smulvih2 committed 5a5318fd on 5.4.x 
    Issue #3562743 by joseph.olstad, smulvih2: Update migration labels and...
smulvih2’s picture

smulvih2
Location Canada 🍁
commented

@joseph good catch on this, nice to bring this into alignment with current core versions. I have made this change to the 5.4.x branch, will include your patch on 6.1.x later.

smulvih2’s picture

smulvih2
Location Canada 🍁
commented
Status: Needs review » Fixed

This has been merged with 6.1.x.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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

joseph.olstad’s picture

joseph.olstad
Primary language French
commented
Version: 6.1.x-dev » 6.2.x-dev
Status: Closed (fixed) » Active
StatusFileSize
new wxt_install.png19.14 KB

somehow this wasn't brought into 6.2.x?

See screenshot from a 6.2.x installation

wxt 6.2.x installation screenshot with gc-intranet theme

We should see Drupal 11 here not Drupal 9

smulvih2’s picture

smulvih2
Location Canada 🍁
commented

Ahh good catch, this must have been merged with 6.1.x when 6.2.x was already created and didn't merge into 6.2.x. Will do this next, thanks!

joseph.olstad’s picture

joseph.olstad
Primary language French
commented
StatusFileSize
new drupal9_in_wxt62x.png11.03 KB

I just checked the 6.2.x branch. In fact, this was never fixed in 6.1.x either. Still says Drupal 9 in 6.1.x/6.2.x branches.

See this:

screenshot of drupal 9 text in the migration

joseph.olstad’s picture

joseph.olstad
Primary language French
commented

in 5.6.x it's Drupal 10, for 6.2.x/6.3.x it should be Drupal 11

smulvih2’s picture

smulvih2
Location Canada 🍁
commented

It was merged to 6.1.x in GutHub - https://github.com/drupalwxt/wxt/blob/275beb98691d32c3d5f0d4bb4e445dc6ffb275b6/modules/custom/wxt_ext/wxt_ext_migration/config/install/migrate_plus.migration.wxt_menu_link.yml#L62

But missing from 6.1.x in drupal repo, will fix this now.

smulvih2’s picture

smulvih2
Location Canada 🍁
commented
StatusFileSize
new wxt-6.1.5.png163.41 KB

Actually, we were preparing for 6.1.5, so the changes above the highlighted commit were in GitHub but not drupal repo. THen we released 6.2.x and 6.1.x got dropped.

So now we just need to merge with 6.2.x.

WxT 6.1.5 prep

smulvih2’s picture

smulvih2
Location Canada 🍁
commented

Ok I added the missing commits that were prepared for 6.1.5 but never made it, and never made it to 6.2.x. Pushed to GitHub and Drupal repo to trigger builds.

smulvih2’s picture

smulvih2
Location Canada 🍁
commented
Status: Active » Fixed

Ok this has been pushed to 6.2.x and tests are passing in GitHub and Drupal repo. This will be included in next release 6.2.1.

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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