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.

Sign up for GitHub

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

Jump to bottom

Latest Changes #526

Merged
merged 29 commits into from
Sep 11, 2024
Merged

Latest Changes #526

merged 29 commits into from
Sep 11, 2024

Conversation

oakdbca
Copy link
Member

@oakdbca oakdbca commented Sep 6, 2024 •
edited
Loading

OCR Cleanup

  • Remove unused OCR fields / models

OCR Bulk Import Tool

  • Add ability to exclude certain fields from the list on a per model basis
  • Filter out models from the content type dropdown if all their fields are already included in the schema
  • Make Selected Column Details panel sticky top so it's still visible when scrolling down when there are lots of columns
  • Add full traceback to log output when bulk import encounters unexpected exception
  • Allow adding all fields for whole model or just mandatory fields for whole model at once rather than having to select single columns
  • Importer is now successfully importing multiple models per row (Tested OCR with OCRVegetationStructure)
  • Importer is now successfully associating a species with the OCR based on a lookup that spans multiple models (i.e. taxonomy__scientific_name

@oakdbca
Remove unused document field from OccurrenceReportReferral.
6f2f009
@oakdbca
Remove unused model OccurrenceReportReferralDocument
50b1e58
@oakdbca
Bug fix: In the case of completing a referral, the geometry_data is a…
2f6ff56 
…lready a dict so added code to be able to cope with such a scenario.
@oakdbca
Remove unused field referral_text from OccurrenceReportReferral model.
c8c7e18
@oakdbca
Remove unused sent_from field from OccurrenceReportReferral model.
101d4fb
@oakdbca
Remove unused boundary field from OCRLocation.
9ccea6a
@oakdbca
Remove unused new_occurrence field from OCRLocation model.
f1ceda0
@oakdbca
Remove unused intersects field from OccurrenceReportGeometry and Occu…
d8d857b 
…rrenceGeometry models.
@oakdbca
Remove unused boundary field from OCCLocation model.
cdfe8b1
@oakdbca
Remove references to fields that have been removed.
2470fba
@oakdbca
Remove references to unused fields.
ecc7b74
@oakdbca
Remove more references to intersects field.
a3b83af
@oakdbca
Remove unused fields review_date and reviewed_by from Occurrence model.
cafb084
@oakdbca oakdbca changed the title OCR Field Cleanup OCR / OCC Field Cleanup Sep 6, 2024
@oakdbca
Bug fix. Add allow_null and required false to id field on OccurrenceR…
95a4088 
…eportBulkImportSchemaColumnNestedSerializer. As was giving false positives for validation.
@oakdbca
Add code to allow adding all fields for whole model or just mandatory…
b64b732 
… fields for whole model. Restucture page so that the selected colun details are sticky top (i.e. when scrolling down when there are lots of columns you can still see the column details).
@oakdbca
Bug fix: Check for selectedColumn id instead of selected content type…
1ed3d79 
… id when deciding if showing options is necessary.
@oakdbca
Add BULK_IMPORT_EXCLUDE_FIELDS class variable to OccurrenceReport model.
e16f305
@oakdbca
Exclude content types if they are listed in the model's BULK_IMPORT_E…
9884496 
…XCLUDE_FIELDS variable.
@oakdbca
Filter out models from the content type dropdown if all their fields …
85289ad 
…are already included in the schema.
@oakdbca
Simplify html for schema details section.
802ade6
@oakdbca
Get some more details when logging locally.
04770cb
@oakdbca
Make sure to get traceback when handling unnamed exceptions.
c155556
@oakdbca
Work to get child records importing successfully.
481dbc8
@oakdbca
Add django_lookup_field_name field to OccurrenceReportBulkImportSchem…
677c121 
…aColumn model.
@oakdbca
Make the django_lookup_field_name field default to 'id'.
35be65d
@oakdbca
Add lookup field options as an attribute for fields that are related.
06ceadd
@oakdbca
Code to allow selection of django lookup field or entry of custom fie…
8995a09 
…ld for advanced uses. Also bug fix that was breaking popovers when the first column was selected.
@oakdbca
Add function that returns the display field for a model.
ab33b5a
@oakdbca
Column validation now also performs the foreign key lookup when neces…
aa45c35 
…sary and replaces the cell value with an instance of the associated model.
@oakdbca oakdbca changed the title OCR / OCC Field Cleanup Latest Changes Sep 11, 2024
xzzy
xzzy approved these changes Sep 11, 2024
View reviewed changes
Copy link
Collaborator

@xzzy xzzy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sensitivity Check Completed

@xzzy xzzy merged commit a44dfcd into dbca-wa:main Sep 11, 2024
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xzzy xzzy xzzy approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@oakdbca @xzzy