-
Notifications
You must be signed in to change notification settings - Fork 0
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
Ki map #11
Conversation
No broken urls! 🎉 |
Re-rendered previews from the latest commit:
Updated at 2024-02-01 with changes from fdb0436 |
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.
LGTM! I'm guessing the <sup>
will be superseded by OTTR refs in #13
Can you make sure this is up to date with main
?
The widget is not showing up - does it work locally? when you run bookdown::render_book('index.Rmd')
No spelling errors! 🎉 |
@ehumph @avahoffman new commit uses iframe and the embed this map url from google maps. It also renders on my local copy |
Purpose/implementation Section
What changes are being implemented in this Pull Request?
Embedding a sampling map
What was your approach?
Using
knitr::include_url()
to embedWhat GitHub issue does your pull request address?
Issue #7