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

[DoctrineBridge] add missing UPGRADE notes for #50689 and update tests #50699

Conversation

alli83
Copy link
Contributor

@alli83 alli83 commented Jun 18, 2023

Q A
Branch? 6.3
Bug fix? no
New feature? no
Deprecations? yes
Tickets
License MIT
Doc PR

$isSameDatabase parameter was introduced in #48059. It has been added in order to know whether the database used by doctrine is the same as the one used by the component when integrating the latter with doctrine migrations.

Also related to #50689

@nicolas-grekas
Copy link
Member

Thank you @alli83.

@nicolas-grekas nicolas-grekas merged commit 8648df5 into symfony:6.3 Jun 21, 2023
7 of 9 checks passed
nicolas-grekas added a commit that referenced this pull request Jun 26, 2023
…eDatabase arg… (alli83)

This PR was merged into the 6.3 branch.

Discussion
----------

[DoctrineBridge] add missing changelog mention for isSameDatabase arg…

| Q             | A
| ------------- | ---
| Branch?       | 6.3
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Tickets       |
| License       | MIT
| Doc PR        |

$isSameDatabase parameter was introduced in #48059. It has been added in order to know whether the database used by doctrine is the same as the one used by the component when integrating the latter with doctrine migrations.

Also related to #50699 (UPGRADE mention) and #50689

Commits
-------

208861a [DoctrineBridge] add missing changelog mention for isSameDatabase arg introduced in #48059
nicolas-grekas added a commit that referenced this pull request Jun 28, 2023
… $isSameDatabase to configureSchema() methods (alli83)

This PR was merged into the 7.0 branch.

Discussion
----------

[Cache][DoctrineBridge][Lock][Messenger] Add parameter $isSameDatabase to configureSchema() methods

| Q             | A
| ------------- | ---
| Branch?       | 7.0
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Tickets       |
| License       | MIT
| Doc PR        |

$isSameDatabase parameter was introduced in #48059.
This PR aims to remove BC layer on configureSchema method and $isSameDatabase parameter

This parameter has been added in UPGRADE-6.3 in #50699

Commits
-------

7a8a8bf [DoctrineBridge]rm BC layer configureSchema check database
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants