Skip to content

Conversation

@mjauvin
Copy link
Member

@mjauvin mjauvin commented Oct 15, 2024

This allows setting SQLite db path relative to app root, even when running with mirrored install.

@mjauvin mjauvin added this to the 1.2.7 milestone Oct 15, 2024
@mjauvin mjauvin requested a review from LukeTowers October 15, 2024 21:24
@mjauvin mjauvin self-assigned this Oct 15, 2024
@LukeTowers LukeTowers changed the title Override SQLiteConnector to fix path resolution Support project relative paths to sqlite databases regardless of webroot Oct 16, 2024
@LukeTowers LukeTowers merged commit 752c162 into develop Oct 16, 2024
10 checks passed
@LukeTowers LukeTowers deleted the improve-sqlite-path-resolution branch October 16, 2024 08:56
taylorotwell added a commit to laravel/framework that referenced this pull request Feb 6, 2025
* Support relative paths to SQLite databases

Related: #45626, wintercms/storm#191

* Styling

* Update SQLiteConnector.php

---------

Co-authored-by: Taylor Otwell <taylor@laravel.com>
taylorotwell added a commit to illuminate/database that referenced this pull request Feb 6, 2025
* Support relative paths to SQLite databases

Related: laravel/framework#45626, wintercms/storm#191

* Styling

* Update SQLiteConnector.php

---------

Co-authored-by: Taylor Otwell <taylor@laravel.com>
@mjauvin
Copy link
Member Author

mjauvin commented Feb 8, 2025

@LukeTowers should we revert this in the Laravel 11 branch since the fix has been merged upstream?

@LukeTowers
Copy link
Member

@mjauvin yes please

mjauvin added a commit that referenced this pull request Feb 8, 2025
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.

3 participants