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

Always use mapped type #581

Merged
merged 2 commits into from
Dec 9, 2024
Merged

Always use mapped type #581

merged 2 commits into from
Dec 9, 2024

Conversation

johnnyomair
Copy link
Collaborator

MikroORM will interpret the provided type as column type if no mapped type alias is known:

The type option here allows several input forms. We are using the text type name here, which is mapped to the TextType - a mapped type representation used internally by the ORM. If you provide a string value there, and it won't match any known type alias, it will be considered as the column type.
MikroORM docs

MikroORM will interpret the provided type as column type if no mapped type alias is known:
> The type option here allows several input forms. We are using the text type name here, which is mapped to the TextType - a mapped type representation used internally by the ORM. If you provide a string value there, and it won't match any known type alias, it will be considered as the column type.
> – https://mikro-orm.io/docs/guide/first-entity#scalar-properties
thomasdax98
thomasdax98 previously approved these changes Dec 3, 2024
dkarnutsch
dkarnutsch previously approved these changes Dec 3, 2024
nsams
nsams previously approved these changes Dec 4, 2024
@johnnyomair johnnyomair dismissed stale reviews from nsams, dkarnutsch, and thomasdax98 via 75c76db December 4, 2024 11:53
@johnnyomair johnnyomair merged commit 707347d into main Dec 9, 2024
3 checks passed
@johnnyomair johnnyomair deleted the always-use-mapped-type branch December 9, 2024 09:07
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.

5 participants