Closed kevinfunk closed 6 years ago
@briancwald can you give this a quick gander and let us know if it meets the theme css/sass/comments requirements and is functional on your end?
Yeah this all looks fine - not sure why the lock needed to be updated here, but if it's not an issue than its fine to merge that too.
Yeah we updated the lock because tests were failing otherwise.
On Wed, Apr 18, 2018 at 7:01 AM, Brian notifications@github.com wrote:
Yeah this all looks fine - not sure why the lock needed to be updated here, but if it's not an issue than its fine to merge that too.
— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/acquia/df/pull/74#issuecomment-382362754, or mute the thread https://github.com/notifications/unsubscribe-auth/ABDTPurD3xG4_cNrIl4QsXlvY-ddCNrFks5tpysHgaJpZM4TYw6b .
--
Brant WynnTechnical Lead, Presales | Acquia Mobile: +1 312-607-9430 | Certifications https://certification.acquia.com/registry?fname=Brant&lname=Wynn
DF-1528 Embedding a Youtube video in a block will take it out of bounds