Survey metadata

[beamalsky]

Overview

Connects #63. Creates elements for survey metadata. Related issues that came out of this but should be addressed separately: #88; #91; #92; #93

To Do

Add fields on survey model:

• location
• study

Add form elements:

• total
• time_start
• time_stop
• time_character
• representation
• microclimate
• temperature_c
• method

Other things:

• We'll need observational surveys to require a Total form element. And maybe time_start, time_stop, method, and representation. Open a new issue to handle these. (update: issue Pick a method for handling PLDP required fields #88)
• Get data handler working with all of this
• Figure out relationship between time_stop and submission time (update: issue Create separate time_stop and time_submission fields in django-pldp #91)

Testing Instructions

• If you don't already have it, create the database: pg_restore -C -j4 --no-owner just-spaces.dump | psql
• Migrate: python manage.py migrate
• Login as testuser (password in the DataMade LastPass under "Just Spaces Staging")
• Create a new survey
• Add some of the metadata fields
• Publish your survey
• Submit it. Do the fields save as you'd expect on the survey model?

Connects #63

[jeancochrane]

Couple small questions, but I think we'll be able to get this over the finish line today.

[jeancochrane]

Is this change just because the test settings weren't changing anything? Do the tests not use a separate DB?

[beamalsky]

Yep, they were the same and it seemed like an unnecessary hassle to have to change settings in two places.

[jeancochrane]

One small question here -- either the dedent I pointed out is a typo which you can fix quickly, or there's a semantic reason for it and it can stay. Either way, feel free to merge once you've made that call!

[jeancochrane]

I would expect this to be a method on the class too, but it looks like it's dedented. Is there a reason for that?

[beamalsky]

This is a holdover from Fobi! I don't see a need to mess with it, since it works and I'm not sure where else it's used in the package.

[jeancochrane]

Sounds great, thanks for clarifying.