-
Notifications
You must be signed in to change notification settings - Fork 4
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
Put tile urls for datasets #623
Conversation
✅ Deploy Preview for veda-ui ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
1194307
to
cde52f1
Compare
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.
This was a great base to start. 🎉 I rebased the work after merging the coordinates PR. (be sure to reset your branch)
- I noticed that the xyz endpoint was actually the tilejson, which may not be exactly what we want, so I fetched and stored the xyz url.
- I did not change anything about the wmts as I'm not sure about the format.
- Added support for the vector data tile endpoint
- Tweaked the widget styles and the different types
I think this is good to go! (maybe double check the wmts)
app/scripts/components/common/mapbox/layers/raster-timeseries.tsx
Outdated
Show resolved
Hide resolved
252f494
to
d381a05
Compare
@anayeaye brought up a good point that custom color formula can make it look like it is not working (ex. color_formula=gamma+r+1.05.). I don't think this should be a blocker, but worth noting. Thanks for testing the features 🙇 🙇 |
This PR is based on : #620
Close #594