-
Notifications
You must be signed in to change notification settings - Fork 10
#41790 documentation additions #8
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
Conversation
…t the top rather than the steps to install using Composer. Added section about development and contribution.
|
Hi @madsnorgaard Changes you are proposing will work only for docksal environment and will not helpful for setting up project without it. |
|
The problem with the current instructions is that they do not work. We propose to replace with the docksal instructions for now and then we can reinstate the LAMP instructions once you have this method working. It is best practice to have working instructions in the README. If you already have working instructions for the installation using LAMP stack, we propose that this is added above the Docksal instructions. |
|
I see. I agree that current "Usage" section is outdated and need to be updated. |
|
What would the steps be to install this on LAMP stack? It would be really beneficial to know these steps as well. Currently Problem 1
|
|
We have received feedback from @lats1 and ITK on these changes. I believe this is solid documentation for users of Docksal and traditional install. This should with review from Bellcom be ready to merge into develop. |
Could you add these steps for LAMP usage? The composer problem you have in related to missing php extension in your environment
When you get installed this extension you will not get this error. |
Adding the procedure to install using Docksal first on the page rather than how to install using Composer, which currently no longer works.