Repository navigation
Mixed HTTP and HTTPS cases #9
Description
Test for @tsamaya 's PR #8 to allows https.
If only warning, we can marge it and will fix other issues by next patch.
-
D3, Turf, OSM
@tsamaya have done them already.
-
CartoDB Basemap
I think that it is ignored at this time.
Esri is better. But it should not update and add plugin for just a basemap.
Stamen is also invalid? https://c.tile.stamen.com/toner/11/328/790.png
cartodb-basemaps-{s}.global.ssl.fastly.net can be accessed via https. -
Open Data Portal of Esri Japan
Yes, I agree. The geojson will be embedded into map-effects-100/data/.
-
Font Awesome
I think that it is ignored at this time.
A known issue -> Unexpected CSS token softlayer/sl-ember-components#1034 -
Video from mazwai
http://mazwai.com/
HTTPS is not supported.
We should take a large file into other storage services. S3?
Activity
[-]Mixed HTTPS and HTTPS cases[/-][+]Mixed HTTP and HTTPS cases[/+]ynunokawa commentedon Dec 6, 2015
Basemap tiles of CartoDB and OSM is accessed via http.
However they show warning messages.
Current
Better solution
Anything else?
tsamaya commentedon Dec 6, 2015
there is also
<script src="http://d3js.org/d3.v3.min.js"></script><script src="http://api.tiles.mapbox.com/mapbox.js/plugins/turf/v2.0.0/turf.min.js"></script>I can update my branch linked to the PR so it will automatically added to the PR
ynunokawa commentedon Dec 6, 2015
Your work is fast and amazing! 👍
You can update them?
Thanks a lot!
tsamaya commentedon Dec 6, 2015
I work on it. I will update it
tsamaya commentedon Dec 6, 2015
done except about geojson loaded from Open Data Portal of Esri Japan. the SSL certificat gives me a warning.
tsamaya commentedon Dec 6, 2015
certificat from cartocdn.com also gives me a warning. ssl is not valid (certicat belongs to a.ssl.fastly.net)
we can change to tiles to another provider (stamen has a black and white map; esri dark background ; other one ?)
the geojson can be embedded into the repo.
ynunokawa commentedon Dec 7, 2015
D3, Turf, OSM
CartoDB Basemap
Open Data Portal of Esri Japan
Font Awesome
It is BETTER that the samples of the repo is supported HTTPS.
And It is OK not to change the cases that show a warning message at this time.
It is an example. I think that it should not change UI rashly.
If adding the geojson into the repo and updating the links, I try to marge the PR.
After marge it, we will continue to face this issue (the links in README use http).
tsamaya commentedon Dec 7, 2015
Hi @ynunokawa,
http://jsbin.com/poriso/edit?html,js,output
It is nice to work on it, thank you for this repo !
ynunokawa commentedon Dec 7, 2015
Awesome! 👍
Stamen is better for everyone to use our examples because it is under CC BY 3.0.
If using Esri Basemap, we want to propose some ideas to use the feature of Esri Leaflet.
tsamaya commentedon Dec 9, 2015
Hi, I found cartoDB tiles working on both schemes : http and https.
http://tsamaya.github.io/map-effects-100/Leaflet/
https://tsamaya.github.io/map-effects-100/Leaflet/
🚀
So I added a commit on my PR #8
ynunokawa commentedon Dec 10, 2015
I have confirmed them, excellent!
Could you also commit to embed the GeoJSON from Open Data Portal into the repo?
tsamaya commentedon Dec 10, 2015
I named the file coming fomr OpenDataPortal Shizuoka_100_waterside.geojson as I don't really understand the name coming from the portal : 静岡県_静岡のみずべ100選.geojson
Commit done right now 🎆
ynunokawa commentedon Dec 10, 2015
Good! there is no problem.
I have marged your PR!!
Thanks a lot 😄 😄 😄