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

Update forge docs to recommend disabling toggles #1572

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Technobabble17
Copy link

A more standard install might recommend not running migrations or composer installing during the install repo stage.

I added a best practice tip, and updated the image with a current and compressed snapshot.

I didn't include anymore, but it might be great to discuss how to setup the environment variables afterwards as well as generating the deploy key!

@jasonvarga
Copy link
Member

Why would you need to disable those?

@Technobabble17
Copy link
Author

Using statamic v5 and cli 3.4.1, when installing a repo in Forge, the install will error out, even when setting the environment tab ahead of time.

...
Generating optimized autoload files
> Illuminate\Foundation\ComposerScripts::postAutoloadDump
> @php artisan package:discover --ansi

In Connection.php line 825:
                                                                               
  Database file at path [/home/forge/forge-test.com/database/database.sqlite]  
   does not exist. Ensure this is an absolute path to the database. (Connecti  
  on: sqlite, SQL: select exists (select 1 from sqlite_master where name = 's  
  ites' and type = 'table') as "exists")                                       
                                                                               
In SQLiteConnector.php line 37:
                                                                               
  Database file at path [/home/forge/forge-test.com/database/database.sqlite]  
   does not exist. Ensure this is an absolute path to the database.            
                                                                               
Script @php artisan package:discover --ansi handling the post-autoload-dump event returned with error code 1

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.

2 participants