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

[Bug] Have html export always use latest published version #2292

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

easherma
Copy link

Signed-off-by: Eric Sherman [email protected]

fixes #2028 , fixes #2111, fixes #2202

This previous commit:
dbba7da#diff-d487565baf71f3a3554594403f1de25ce00ba6b634da5248f461f14dbddfe5c2R21 updated the version of kepler.gl to a build that is not published to unpkg.com

To fix this problem and avoid in the future, I removed the specific version tag. According to https://unpkg.com/ , this means the latest published version will always be used (currently 2.5.5)

Thanks for the great project!

This previous commit:
keplergl@dbba7da#diff-d487565baf71f3a3554594403f1de25ce00ba6b634da5248f461f14dbddfe5c2R21 updated the version of kepler.gl to a build that is not published to unpkg.com

To fix this problem and avoid in the future, I removed the specific version tag. According to https://unpkg.com/ , this means the latest published version will always be used (currently 2.5.5)
@easherma
Copy link
Author

Sorry, I put the DCO in the PR but not the commit message. LMK if this blocks approval and I can reword the message.

@@ -18,7 +18,7 @@
<script src="https://unpkg.com/[email protected]/dist/styled-components.min.js" crossorigin></script>

<!-- Load Kepler.gl-->
<script src="https://unpkg.com/kepler.gl@3.0.0-alpha.0/umd/keplergl.min.js"></script>
<script src="https://unpkg.com/kepler.gl/umd/keplergl.min.js"></script>
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Q: Have you confirmed if this load the latest 2.x production version or the latest alpha version? I am assuming the alpha versions are published with a tag and I seem to remember that one needs to specify the tag in the unpkg url in such cases?

I assume we want this to load the same version that is hosted on kepler.gl/demo.

@igorDykhta Any thoughts?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When I tested this locally it loaded 2.5.5. I also browsed the available versions at https://unpkg.com/browse/kepler.gl/

I can see alpha tags for previous versions but not 3.0.0. https://kepler.gl/demo/ appears to be all compiled inside a bundle.js , so I couldn't double check the version (though I see it on the browser on the left-hand side)

My thinking was that this change would mean as soon as 3.0 is published it would be used, and in the meantime the latest version would be used which seems to be the workaround working for most folks based on past issues filed.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

confirming this works if you change the version of kepler.gl to load:

<script src="https://unpkg.com/[email protected]/umd/keplergl.min.js" crossorigin></script>

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants