Skip to content

Conversation

timgraham
Copy link
Collaborator

"embeddedDocuments" isn't a BSON type.

@timgraham
Copy link
Collaborator Author

@WaVEV I guess this is a user facing bug fix since "document" should be used for embedded model search indexes, while embeddedDocuments is only for arrays of documents. Correct? If so, I will create a ticket and add a release note.

@WaVEV
Copy link
Collaborator

WaVEV commented Oct 4, 2025

@WaVEV I guess this is a user facing bug fix since "document" should be used for embedded model search indexes, while embeddedDocuments is only for arrays of documents. Correct? If so, I will create a ticket and add a release note.

Yes, It should have the same type as JSON. So embedded documents ins't correct.

@timgraham timgraham changed the title Correct EmbeddedModelField.db_type() to "object" INTPYTHON-778 Correct Polymorphic/EmbeddedModelField search index type Oct 6, 2025
Also, correct EmbeddedModelField.db_type() to "object".
("embeddedDocuments" isn't a BSON type).
@timgraham timgraham merged commit 2c1409e into mongodb:main Oct 6, 2025
17 checks passed
@timgraham timgraham deleted the emf-db-type branch October 6, 2025 14:06
aclark4life added a commit to aclark4life/django-mongodb-backend that referenced this pull request Oct 6, 2025
aclark4life added a commit to aclark4life/django-mongodb-backend that referenced this pull request Oct 6, 2025
- Add mongocryptd to atlas CI
- Doc updates
- Remove custom db_type, not needed after mongodb#414
- SupportsQueryableEncryptionTests require MongoDB 8
- Update test model names prefix and suffix
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.

2 participants