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

Picking up trees from a year ago tried to load all of SF as geojson b… #134

Merged
merged 1 commit into from
Mar 8, 2023

Conversation

zoobot
Copy link
Member

@zoobot zoobot commented Mar 8, 2023

…ecause we modified ids in 2022 june

@zoobot
Copy link
Member Author

zoobot commented Mar 8, 2023

Temporarily fixes #133
We need to deal with the tree-sources repo picking up all the trees modified in the db when the city is NOT in sources.
Better fix will be to add city and source at the same time as adding the first tree in a city.

@zoobot zoobot requested a review from mwpark2014 March 8, 2023 07:56
@zoobot zoobot marked this pull request as ready for review March 8, 2023 07:56
@zoobot zoobot merged commit c2940e4 into main Mar 8, 2023
zoobot added a commit that referenced this pull request Mar 14, 2023
@zoobot zoobot deleted the zoobot/bug/load-spike branch March 14, 2023 19:10
zoobot added a commit that referenced this pull request Mar 20, 2023
* move broken url field to end for FE checkbox in data

* fix merge conflicts

* sources

* zoobot/feature/data-sources

* change id_name to id_source_name

* Revert "fix merge conflicts"

This reverts commit 6cfd267.

* sources changes

* sources massaging

* sources fixes

* sources

* testing changes

* fixed post after nested crosswalks

* remove console and jest

* Add all cities not explicitely specified for sources download (#128)

* Add all cities not explicitely specified for sources download

* lengthen time for geojson for temporarily

* Picking up trees from a year ago tried to load all of SF as geojson because we modified ids in 2022 june (#134)

* Add GET tests for sources=all and sources array

* fixed security issues

* code review cleanup

* removed find cities and countries as we are not currently using

* change queries to prepared statements
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.

None yet

1 participant