Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix: Update composer scripts and tailwind dependencies #1243

Closed
wants to merge 2 commits into from

Conversation

nialma
Copy link

@nialma nialma commented Oct 31, 2024

Winter CMS Build
dev-develop

PHP Version
8.2.24

Database engine
MySQL/MariaDB

Plugins installed
No response

System
Windows 11

### This PR fixes issue #1242 and replaces the deleted PR #1241.

Changes:

  1. composer.json:

    • Modified post-create-project-cmd to stop after copying files
    • Prevents automatic installation before key:generate
    • Allows manual control over the installation process
    • Makes the installation process more reliable
  2. modules/cms/console/scaffold/theme/tailwind/package.stub:

    • Updated devDependencies for tailwind theme
    • Ensures correct dependencies are installed during theme creation

The main benefit is that the installation process now stops at the right moment, allowing for proper key generation before continuing with the installation.

Related:

@bennothommo
Copy link
Member

bennothommo commented Oct 31, 2024

@nialma per this comment, I don't think we should rollback the post-create-project-cmd changes as you've suggested in this PR, and instead we should modify it to work non-interactively. I'd be happy for you to submit another PR with just the Tailwind depdency updates however.

@nialma
Copy link
Author

nialma commented Oct 31, 2024

@bennothommo The problem I see with the fully automatic installation is that there are different database types that differentiate database names and sometimes the users.....Therefore, I currently see the effort as being too great to fully automate the installation.....

I'm currently working on ensuring that when you create a Tailwind theme via the backend, mix is ​​automatically installed and configured...

First of all without alpinejs... that comes afterwards....

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.

Composer post create project commands fail on Windows
2 participants