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

Thank you for upgrading this awesome boilerplate starter for Slim 4: Some issues I noticed FYI only. #19

Open
markjcorrigan opened this issue Feb 14, 2024 · 0 comments

Comments

@markjcorrigan
Copy link

markjcorrigan commented Feb 14, 2024

<script src="/assets/libs//jquery.min.js"></script> //needed to remove //

i.e. <script src="/assets/libs/jquery.min.js"></script>
2.
package.json

npm{
"name": "slim4",
"version": "1.1.0",
"description": "Slim4 boilerplate",
"main": "webpack.mix.js",
"scripts": {
"dev": "node node_modules/cross-env/dist/bin/cross-env.js NODE_ENV=development node_modules/webpack/bin/webpack.js --progress --config=node_modules/laravel-mix/setup/webpack.config.js",
"watch": "node node_modules/cross-env/dist/bin/cross-env.js NODE_ENV=development node_modules/webpack/bin/webpack.js --watch --progress --hide-modules --config=node_modules/laravel-mix/setup/webpack.config.js",
"watch-poll": "node node_modules/cross-env/dist/bin/cross-env.js NODE_ENV=development node_modules/webpack/bin/webpack.js --watch --watch-poll --progress --hide-modules --config=node_modules/laravel-mix/setup/webpack.config.js",
"hot": "node node_modules/cross-env/dist/bin/cross-env.js NODE_ENV=development node_modules/webpack-dev-server/bin/webpack-dev-server.js --inline --hot --config=node_modules/laravel-mix/setup/webpack.config.js",
"prod": "node node_modules/cross-env/dist/bin/cross-env.js NODE_ENV=production node_modules/webpack/bin/webpack.js --progress --hide-modules --config=node_modules/laravel-mix/setup/webpack.config.js"
},

a) I removed npm at top left as complaint about json not being able to run in multiple levels
b) I removed --hide modules as complaint when I ran npm run dev
c) I updated the dependencies as mix would not compile when I used npm run dev.
Below worked.
"homepage": "https://github.com/FernandoCalmet",
"devDependencies": {
"cross-env": "^5.2.1",
"laravel-mix": "^6.0.49",
"resolve-url-loader": "^5.0.0",
"sass": "^1.19.0",
"sass-loader": "^14.1.0",
"vue-template-compiler": "^2.7.16"
}
}
I.e. laravel mix came back compiled successfully when I ran npm run dev.

  1. I initially removed captcha as I struggled to get it configured. I noticed that when I tried to register a new user it failed.
    I needed to comment out captcha on the registration form.
    Also commented out // 'g-recaptcha-response'=> 'required|recaptcha' in the AuthController.
    This done I could register a new user.
    I will replace captcha when I figure out how it works.
    I must confess that I have never used a captcha before so new to me.

I have an issue with the Database/Migrations/MigrationsStub.php?
In PHPStorm it is complaining with a red line from app/Databse/Migrations/MigrationStub.php. Not sure why. But program is working regardless.

Thank you for upgrading the boiler plate to work on php version 8+. Awesome!

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

No branches or pull requests

1 participant