-
Notifications
You must be signed in to change notification settings - Fork 46
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
Array Field is not supported #7
Comments
@rvinzent Kindly think about this and let me know if this can be achieved. I'll try to help if possible |
@yvsssantosh apologies for the delayed response. This sounds like a nice addition to the project 😄 . It should come with some checking to make sure array fields are supported by the database. |
I think best way to implement this without building a complex logic to check whether database supports array fields or not will be to do it the way Django does.
from dynamic_models import fields as dynamic_fields
color_field_schema = FieldSchema.objects.create(name='color', data_type=dynamic_fields.CharField(max_length=200)')
from dynamic_models.postgres.fields import ArrayField This way, it's explicit and devs can make their own decision. I know it's already implemented and will be release in 0.2.0 but I just wanted to give my suggestion. Thanks for this lib |
Where can you see that it is already implemented and that there is a version 0.2.0? |
@dankgen-tobias |
@arpitremarkable @dankgen-tobias Increased data type support is absolutely on the radar, and PRs are always appreciated! |
@arpitremarkable I like the idea though and it should be possible to implement and allow a lot more flexibility to the kinds of fields that can be used. |
Requesting feature to support type ArrayField to allow the user to store data in an array.
The text was updated successfully, but these errors were encountered: