In the upgrade message it says:

... the Nodesquirrel functonality as been disabled.

I changed it to has.

Also need to fix "functonality", "NodeSquirel" and for good measure, make sure that the human readable name is consistent (i.e. "NodeSquirrel").

Please review.

CommentFileSizeAuthor
#5 backup_migrate-spelling_typo_in_backup_migrate_update_7310-3119038-5.patch1.78 KBgisle
#4 backup_migrate-typo-fix-3119038-4.patch1.18 KBsolideogloria
backup_migrate-typo-fix.patch1.18 KBsolideogloria

Comments

solideogloria created an issue. See original summary.

solideogloria’s picture

solideogloria commented
Issue summary: View changes
gisle’s picture

gisle
he/him
Primary language Norwegian Bokmål
Location Norway
commented
Status: Needs review » Needs work

There is still a typo in the patch: "functonality" should be "functionality".

solideogloria’s picture

solideogloria commented
Status: Needs work » Needs review
StatusFileSize
new backup_migrate-typo-fix-3119038-4.patch1.18 KB

Fixed

gisle’s picture

gisle
he/him
Primary language Norwegian Bokmål
Location Norway
commented
Assigned: solideogloria » Unassigned
Issue summary: View changes
StatusFileSize
new backup_migrate-spelling_typo_in_backup_migrate_update_7310-3119038-5.patch1.78 KB

This patch includes the previous spelling corrections ("as" → "has", "functonality" → "functionality"). It also corrects "NodeSquirel" → "NodeSquirrel" in two different places, as well making sure that the human readable name for NodeSquirrel is consistent (i.e. "NodeSquirrel"). This is the official way to spell it, according to this page: https://pantheon.io/nodesquirrel-service-end-life

solideogloria’s picture

solideogloria commented
Status: Needs review » Reviewed & tested by the community

Looks good to me.

damienmckenna’s picture

damienmckenna
Location TN, USA
commented
Status: Reviewed & tested by the community » Fixed
Parent issue: » #3116785: Plan for Backup and Migrate 7.x-3.8

Thank you both. Committed.

Status: Fixed » Closed (fixed)

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