We currently depend on shepherd.js 9.1.0. Update to shepherd.js 10.0.1 which is the latest release: https://www.npmjs.com/package/shepherd.js. Release notes: https://github.com/shipshapecode/shepherd/blob/master/CHANGELOG.md#v1000...

CommentFileSizeAuthor
#3 3308783-3.patch1002.27 KBlauriii
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

lauriii created an issue. See original summary.

lauriii’s picture

lauriii
he/him
Primary language Finnish
Location Finland
CreditAttribution: lauriii at Acquia commented
Title: Update to shepherd.js 10.0.1 » Update shepherd.js to 10.0.1
lauriii’s picture

lauriii
he/him
Primary language Finnish
Location Finland
CreditAttribution: lauriii at Acquia commented
Status: Active » Needs review
FileSize
3308783-3.patch1002.27 KB

Ran the Tour JavaScript tests locally and they passed 🎉

longwave’s picture

longwave
he/him
Primary language English
Location UK
CreditAttribution: longwave at Full Fat Things commented
Status: Needs review » Reviewed & tested by the community

The breaking change appears not to affect us so this is RTBC.

nod_’s picture

nod_
Primary language French
Location Lille
CreditAttribution: nod_ at Acquia commented

another RTBC cross post :)

  • bnjmnm committed c37ade4 on 10.1.x 
    Issue #3308783 by lauriii, longwave: Update shepherd.js to 10.0.1

  • bnjmnm committed 84d50e3 on 10.0.x 
    Issue #3308783 by lauriii, longwave: Update shepherd.js to 10.0.1
bnjmnm’s picture

bnjmnm
he/him
Primary language English
Location Ann Arbor, MI
CreditAttribution: bnjmnm at Acquia commented
Status: Reviewed & tested by the community » Fixed

Confirmed a local build didn't do anything unexpected, and reviewed the changelog to confirm there are no changes of concern.

Committed to 10.1.x and cherry picked to 10.0.x

Status: Fixed » Closed (fixed)

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