Skip to content

fix: laravel 10.34.0 changed database internals #150

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 7 commits into from
Nov 29, 2023
Merged

fix: laravel 10.34.0 changed database internals #150

merged 7 commits into from
Nov 29, 2023
+123 −24

Conversation

taka-oyama
Copy link
Collaborator

@taka-oyama taka-oyama commented Nov 29, 2023 •
edited
Loading

@taka-oyama taka-oyama added the chore Basic cleanup label Nov 29, 2023
@taka-oyama taka-oyama added this to the 6.1 milestone Nov 29, 2023
@taka-oyama taka-oyama self-assigned this Nov 29, 2023
@halnique
Copy link
Contributor

laravel/framework#49020
The test failed due to this PR change.

@taka-oyama taka-oyama changed the title fix: laravel 10.34.0 changed transaction internals fix: laravel 10.34.0 changed database internals Nov 29, 2023
@taka-oyama
Copy link
Collaborator Author

All failing tests have been fixed.

@taka-oyama taka-oyama merged commit c2782e4 into master Nov 29, 2023
@taka-oyama taka-oyama deleted the fix/match-origin-transaction branch November 29, 2023 08:05
This was referenced Jun 18, 2024
Closed
Merged
dkkoma added a commit that referenced this pull request Nov 19, 2024
@dkkoma
fix: backport #150
2d1d098
dkkoma added a commit that referenced this pull request Nov 20, 2024
@dkkoma
fix: backport #150 and #166 to 5.x branch (#242)
d62cae5 
* fix: backport #150

* fix: backport #166

* fix: phpstan error

* fix: remove broken test

* fix: docs

* fix: github workflow

* fix release date
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kt81 kt81 kt81 approved these changes

@halnique halnique halnique approved these changes

+1 more reviewer

@r4m-ibrohim r4m-ibrohim r4m-ibrohim approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees

@taka-oyama taka-oyama

Labels
chore Basic cleanup
Projects
None yet
Milestone
6.1
Development

Successfully merging this pull request may close these issues.
4 participants
@taka-oyama @halnique @kt81 @r4m-ibrohim