Skip to content

update wagtail to 2.14.1#7523

Merged
Pomax merged 4 commits intomainfrom
update-wagtail-2-14-1
Oct 4, 2021
Merged

update wagtail to 2.14.1#7523
Pomax merged 4 commits intomainfrom
update-wagtail-2-14-1

Conversation

@Pomax
Copy link
Contributor

@Pomax Pomax commented Sep 28, 2021

Closes #7503

STR:

  • checkout main
  • run inv new-env
  • checkout this branch
  • run inv catchup (which will uplift the local testing db)
  • run inv test-python to verify there are no errors

STR part 2:

  • checkout main
  • run inv new-env
  • set up your local db to be a copy of the staging or production database
  • checkout this branch
  • run inv catchup (which will now apply to "real" data)
  • run docker-compose up and poke around the CMS and pages a bit to make sure nothing broke

@TheoChevalier I've added you as reviewer mostly for the localize-ness, I had to updated wagtail-localize-git to 0.10.0 but as far as I can tell that should have been fine.

@mofodevops mofodevops temporarily deployed to foundation-s-update-wag-u0edmc September 29, 2021 16:33 Inactive
@Pomax Pomax temporarily deployed to foundation-s-update-wag-ujg88v September 29, 2021 16:33 Inactive
@MozillaFoundation MozillaFoundation deleted a comment from github-actions bot Sep 29, 2021
@MozillaFoundation MozillaFoundation deleted a comment from github-actions bot Sep 29, 2021
@TheoChevalier
Copy link
Contributor

TheoChevalier commented Sep 29, 2021

@TheoChevalier I've added you as reviewer mostly for the localize-ness, I had to updated wagtail-localize-git to 0.10.0 but as far as I can tell that should have been fine.

Yup, 0.10.0 is only about supporting 2.14, very limited risk here. I’ve already played with 2.14.1 quickly locally, but will do more thorough testing tomorrow, with actual data too. I expect everything to work better, but you never know!

@danielfmiranda
Copy link
Collaborator

Hi Pomax, I tried to take a look at this PR and test it out, but an issue I am experiencing with the inv copy-stage-data command (documented in ticket #7522) is preventing me from testing this fully!

Copy link
Contributor

@TheoChevalier TheoChevalier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Couldn’t find any issue!

@MozillaFoundation MozillaFoundation deleted a comment from github-actions bot Oct 4, 2021
Copy link
Collaborator

@danielfmiranda danielfmiranda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @Pomax just tested and can confirm that everything is working. Approved!

@github-actions
Copy link

github-actions bot commented Oct 4, 2021

This PR introduces visual differences. Click here to inspect the diffs.

@Pomax Pomax merged commit db7c792 into main Oct 4, 2021
@Pomax Pomax deleted the update-wagtail-2-14-1 branch October 4, 2021 23:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update wagtail to version 2.14.1

4 participants