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

Fix cellToChildPos and childPosToCell docs #845

Merged
merged 1 commit into from
Jun 4, 2024
Merged

Conversation

dfellis
Copy link
Collaborator

@dfellis dfellis commented Jun 4, 2024

I noticed while going through the H3 documentation working on the cli that the live examples for the cellToChildPos and childPosToCell don't work because the website is using an out of date version of h3-js. This small PR fixes that (confirmed locally).

@dfellis dfellis self-assigned this Jun 4, 2024
@dfellis
Copy link
Collaborator Author

dfellis commented Jun 4, 2024

What's currently on the website:

Screenshot from 2024-06-04 17-27-40

After the fix:

Screenshot from 2024-06-04 17-30-24

@coveralls
Copy link

Coverage Status

coverage: 98.826%. remained the same
when pulling 0a84c78 on fix-child-pos-docs
into c18cd89 on master.

Copy link
Collaborator

@nrabinowitz nrabinowitz left a comment

Choose a reason for hiding this comment

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

Thanks for the fix!

@@ -6263,6 +6263,14 @@ react-loadable-ssr-addon-v5-slorber@^1.0.1:
dependencies:
"@babel/runtime" "^7.10.3"

"react-loadable@npm:@docusaurus/react-loadable@5.5.2":
Copy link
Collaborator

Choose a reason for hiding this comment

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

Seems a little odd that the lockfile would change here (different yarn version maybe?) but NBD

@dfellis dfellis merged commit efdd514 into master Jun 4, 2024
34 checks passed
@dfellis dfellis deleted the fix-child-pos-docs branch June 4, 2024 23:25
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

4 participants