-
-
Notifications
You must be signed in to change notification settings - Fork 188
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
refactor(map): TypeScriptize Project Data Map files DEV-16 #5558
base: main
Are you sure you want to change the base?
Conversation
…ta-map # Conflicts: # jsapp/js/actions.d.ts # jsapp/js/components/map/map.es6 # jsapp/js/components/map/map.js # jsapp/js/components/map/map.tsx # jsapp/js/components/map/mapSettings.es6 # jsapp/js/components/map/mapSettings.js # jsapp/js/components/map/mapSettings.tsx
# Conflicts: # jsapp/js/actions.d.ts # jsapp/js/components/map/map.js # jsapp/js/components/map/mapSettings.tsx # jsapp/js/dataInterface.ts
I see, I will have to refactor the code a bit. I will move the notification to happen once immediately after upload, and will display error next to the item in the list rather than displaying notification each time. Does this sound good? :) |
@magicznyleszek sounds to me 🆗 perhaps check with Tessa also? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
see above
🗒️ Checklist
<type>(<scope>)<!>: <title> TASK-1234
frontend
orbackend
unless it's global💭 Notes
Changes here:
actions.d.ts
MapSettings
MapColorPicker
component out ofMapSettings
fileautoBind
and listener mixinsFormMap
index.tsx
fileremoveUnknownLayers
function is doing a cleanupaddNewLayers
is ensuring all different file types end up using the sameonOmnivoreLayerReady
callback instead of relying on things "just working"state.loading
as it wasn't being used for anything really@placemarkio/check-geojson
for validating GeoJSON (omnivore sadly doesn't do it, resulting in crashing UI when invalid GeoJSON is added)@mapbox/leaflet-omnivore
👀 Preview steps
Ideally test this branch with bunch of layer files (map test files.zip)
It's important to test each type, as each type is being handled a bit differently by the code (especially
geojson
andkmz
).Testing an overlay file:
geopoint
question,text
question and bunch of submissionsAfter making above steps, ensure everything works as previously: