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

Added Base64 Image field and PointField #1268

Closed
wants to merge 1 commit into from

Conversation

yigitguler
Copy link

Added PointField and Base64ImageField. I believe that such a framework should support base64 images and should provide a nicer display for POINT fields.

Added PointField and Base64ImageField. I believe that such a framework should support base64 images and should provide a nicer display for POINT fields.
@thedrow
Copy link
Contributor

thedrow commented Dec 4, 2013

Care to add tests and documentation?

@yigitguler yigitguler closed this Dec 4, 2013
@tomchristie
Copy link
Member

Hola @yigitguler - these look neato! In particular I can see the Base64 image field being something that plenty of people would be interested in using. :)

With this sort of thing we're strongly moving in the direction of helping contributors to publish their add-ons as third-party packages that can be separately installed through PyPI, and that are then linked to from the main docs.

I'm writing up some proper contribution docs to guide developers on this right now, and will push up as soon as poss. If you're interested in tackling that I'd be v. willing to help out - this repo https://github.com/dabapps/django-reusable-app is a great place to start, in that it'll get your travis setup, tests and pypi setup file in a decent state.

@yigitguler
Copy link
Author

Hi @tomchristie this looks amazing! Contributing through PyPI wasn't the easiest thing. This will definitely help a lot. I will check this and inform you back when i will create a package for GeoDjango & Rest Framework.

Regards

@tomchristie
Copy link
Member

Thanks! Let us know if you need a hand anytime!

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.

None yet

3 participants