Fix #66, smarter way to detect native field type based on resource field instead of django field

concentricsky / django-tastypie-swagger

An adapter to use swagger-ui with django-tastypie.

Other

132 stars 144 forks source link

Fix #66, smarter way to detect native field type based on resource field instead of django field #72

Closed johnraz closed 10 years ago

johnraz commented 10 years ago

Trying to improve field introspection to allow more db backends compatibility

johnraz commented 10 years ago

Related issue is #66

abdulwahid24 commented 10 years ago

Glad to see this fix, Now tastypie-swagger will have compatibility with MongoEngine.

Seraf commented 10 years ago

@abdulwahid24 did you tried it ? Do you confirm you have no errors anymore ?

abdulwahid24 commented 10 years ago

@johnraz I am sorry, i was assuming that you fixed it after testing but when i tried at my end i am also getting the same error https://github.com/concentricsky/django-tastypie-swagger/issues/73 which @Seraf mentioned.

abdulwahid24 commented 10 years ago

I am sharing my mongo engine project with @johnraz as i found you on bit bucket. @Seraf can you give me your bit bucket username so that i can share this to you also.

johnraz commented 10 years ago

Awesome @abdulwahid24, that will definitely help me reproduce error and test mongoengine projects. I keep this one closed as it didn't introduce any regression (but didn't actually fix #66) ...