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

Naistenhelsinki #141

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

Naistenhelsinki #141

wants to merge 9 commits into from

Conversation

ehaivala
Copy link

Create naistenhelsinki app for displaying location data on a map.

  • Import the naistenhelsinki prototype app
  • Fork react-geoview for frontend
  • Clean and refactor the imported/forked code
  • Open the location data to modal on map marker click
  • Create custom serializer that can serialize Django model object property function values

jussiarpalahti and others added 9 commits September 15, 2017 10:46
Import the prototype app as the starting point for the more finalized version
of `naistenhelsinki` app.
The settings and urls need to be configured to get the `naistenhelsinki`
prototype app actually working with the project.
Remove unused files from naistenhelsinki app to keep the app clean and concise.
Refactor the model classes of naistenhelsinki app. Create page model for the
page that has all the places located on a map. Use OpenStreetMaps for Place
map admin widget and set default coordinates to Helsinki instead of the center
of Atlantic Ocean (0,0 coordinates).

Regenerate all naistenhelsinki migrations because the model renaming.
Fork react-geoview under naistenhelsinki app and configure lightweight webpack
and babel for react app compilation.
Remove unused and commented code and unify code style. Split the components
to their own files for better structure.
No need to capitalize the directory name. Also use `path.join` in webpack
config.
Open detailed place information modal on map place marker click. Implement
custom serializer that can serialize python @Property decorated functions as
some of those values are needed in the modal.
Do some light layout fixes. Move inline css out of templates.
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