-
-
Notifications
You must be signed in to change notification settings - Fork 71
Working with databases #267
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
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR adds comprehensive documentation for working with databases in Yii3, focusing on PostgreSQL setup and migration usage. The guide covers installation, configuration, and basic database operations to help developers get started with database integration.
Key Changes:
- Added PostgreSQL installation instructions (both Docker and manual setup)
- Documented database connection configuration using the
yiisoft/db-pgsqlpackage - Introduced migration setup and usage with practical examples
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 30 out of 30 changed files in this pull request and generated no new comments.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 30 out of 30 changed files in this pull request and generated no new comments.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 38 out of 38 changed files in this pull request and generated 2 comments.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 38 out of 38 changed files in this pull request and generated 5 comments.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 38 out of 38 changed files in this pull request and generated 3 comments.
guide/en/start/databases.md
Outdated
| - [Doctrine](https://www.doctrine-project.org/) | ||
| via [Yii Doctrine package](https://github.com/stargazer-team/yii-doctrine) |
Copilot
AI
Nov 3, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[nitpick] Inconsistent line wrapping. The Doctrine link is split across two lines while similar entries (Yii DB, Yii Active Record, Cycle) are on single lines. For consistency, either keep all entries on single lines or wrap them all similarly.
| - [Doctrine](https://www.doctrine-project.org/) | |
| via [Yii Doctrine package](https://github.com/stargazer-team/yii-doctrine) | |
| - [Doctrine](https://www.doctrine-project.org/) via [Yii Doctrine package](https://github.com/stargazer-team/yii-doctrine) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 38 out of 38 changed files in this pull request and generated 2 comments.
| ## Creating and applying migration | ||
| Now that we have the database, it's time to define the connection. | ||
|
|
||
| Let's use latest versions to be released. Change your `minumum-stability` to `dev` in `composer.json` first. |
Copilot
AI
Nov 3, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Corrected spelling of 'minumum-stability' to 'minimum-stability'.
| Let's use latest versions to be released. Change your `minumum-stability` to `dev` in `composer.json` first. | |
| Let's use latest versions to be released. Change your `minimum-stability` to `dev` in `composer.json` first. |
| POSTGRES_PASSWORD: password | ||
| POSTGRES_USER: user | ||
| volumes: | ||
| - ./runtime/db:/var/lib/postgresql/data:rw |
Copilot
AI
Nov 3, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The volume path ./runtime/db should be ../runtime/db to be consistent with line 50 (- ../runtime:/app/runtime). The current path is relative to docker/dev/ directory but doesn't match the pattern used for other volumes.
| - ./runtime/db:/var/lib/postgresql/data:rw | |
| - ../runtime/db:/var/lib/postgresql/data:rw |
No description provided.