Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

DOCSP-48018: laravel 12 feature compat #3304

Merged
merged 3 commits into from
Mar 5, 2025

Conversation

rustagir
Copy link
Contributor

@rustagir rustagir commented Mar 5, 2025

https://jira.mongodb.org/browse/DOCSP-48018

staging

Checklist

  • Add tests and ensure they pass

@rustagir rustagir requested a review from a team as a code owner March 5, 2025 15:04
@rustagir rustagir requested a review from norareidy March 5, 2025 15:04
@github-actions github-actions bot added the docs label Mar 5, 2025
@rustagir rustagir requested review from a team and GromNaN and removed request for a team March 5, 2025 15:16
Copy link
Contributor

@norareidy norareidy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Comment on lines 67 to 71
* - Multi-database Support / Multiple Schemas
- Partially supported. Laravel uses a dot separator (``.``)
between SQL schema and table names, but MongoDB allows ``.``
characters within collection names, which might lead to
unexpected namespace parsing.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We don't have implemented anything to avoid breaking changes. I would not tell it's partially supported.

@rustagir rustagir merged commit 536327d into mongodb:5.2 Mar 5, 2025
62 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants