Module should delete existing Twig template files

Created on 5 March 2024, 9 months ago

Problem/Motivation

Upon installation, this module changes storage for Twig files from the file system to database (key/value) storage. Existing Twig files are not removed; however.

Steps to reproduce

Start with an existing Site Studio installation.
Install the Acquia Site Studio ACSF module.
Observe template files are still present in public://cohesion/templates.

Proposed resolution

Upon installation, delete the files.
Also, add an update hook for existing installs.

Remaining tasks

Open MR.

User interface changes

None.

API changes

None.

Data model changes

None.

✨ Feature request

Status

Needs review

Version

1.0

Component

Code

Created by

🇺🇸United States Chris Burge

Live updates comments and jobs are added and updated live.

Merge Requests

!3Module should delete existing Twig template files
Open
🇺🇸United States Chris Burge
updated 9 months ago

Comments & Activities

Issue created by @Chris Burge
Merge request !3Delete Twig templates upon install → (Open) created by Chris Burge
Status changed to Needs review 9 months ago3:42pm 5 March 2024
Comment 9 months ago →
🇺🇸United States Chris Burge
Note: Test coverage passes locally; however, since acquia/cohesion is hosted on GitHub (i.e. not on D.O.), I expect that tests will fail on GitLab CI.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024

Module should delete existing Twig template files

Problem/Motivation

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Merge Requests

!3Module should delete existing Twig template filesOpen

Comments & Activities

!3Module should delete existing Twig template files
Open