search

koopjs / koop-cache-memory

Deprecated
Other
0 stars 4 forks source link

Store complete geojson in cache #19

Closed guushub closed 2 years ago

guushub commented 2 years ago

Hi @rgwozdz ,

Had a bit of spare time today to look at the solution for issue #18 and implemented as discussed. I also updated the tests. Had to make sure now that every test gets a 'fresh' geojson and added a check for CRS being present, which conveniently also tests if it's indeed the entire geojson being stored.

Can you review these changes and let me know if I need to adjust anything?

Thanks! Guus

rgwozdz commented 2 years ago

Thanks @guushub! I will take a look today.

rgwozdz commented 2 years ago

Published in @koopjs/cache-memory@1.2.0.

guushub commented 2 years ago

Fantastic. Thanks @rgwozdz !