Merged
Conversation
Member
Author
|
Replaces the PR #7820 |
emenslin
approved these changes
Mar 18, 2026
Collaborator
emenslin
left a comment
There was a problem hiding this comment.
- See that the geography tree is populated.
- See the the chronostartigraphy tree is populated.
- On the tree viewer page, see in the network tab that the request for
leaflet-layers.jsonfile is using/static/config/common/leaflet-layers.jsonand notfiles.specifysoftware.org/specify7/7.8.10/leaflet-layers.json - See that the geography tree is populated.
- See the the chronostartigraphy tree is populated.
Looks good!
CarolineDenis
approved these changes
Mar 18, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add the geography and geologictimeperiod trees to local storage. Instead of accessing default geo tree files remotely, we decided to add them to the repo so that each Specify instance would have the needed files locally. This allows users without internet access to do the important process of setting up default geo locally.
Also note in the implementation that we needed to add local mapping files for the setup preload of Geography and Geologic Time Period trees in order to get them to work.
No new api calls were needed, just using the existing
/static/config/...api now to get the default tree files. This is used, instead of the remote call tofiles.specifysoftware.org/.... We also added theleaflet-layers.jsonto the local config directory, since it was being frequently called tofiles.specifysoftware.org, so now it can be retrieved from/static/config/common/leaflet-layers.json.The files were only a few MB, so compressing the files didn't seem necessary.
Here are the exact file location changes:
https://files.specifysoftware.org/geographyfiles/geonames.csv->config/default_trees/geonames.csvhttps://files.specifysoftware.org/chronostratfiles/GeologicTimePeriod.csv->config/default_trees/GeologicTimePeriod.csvhttps://files.specifysoftware.org/treerows/geography.json->config/default_trees/mapping_files/geography.jsonhttps://files.specifysoftware.org/treerows/geologictimeperiod.json->config/default_trees/mapping_files/geologictimeperiod.jsonhttps://files.specifysoftware.org/specify7/7.8.10/leaflet-layers.json->config/common/leaflet-layers.jsonChecklist
self-explanatory (or properly documented)
Testing instructions
leaflet-layers.jsonfile is using/static/config/common/leaflet-layers.jsonand notfiles.specifysoftware.org/specify7/7.8.10/leaflet-layers.json