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

Update README.md #2

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

Update README.md #2

wants to merge 1 commit into from

Conversation

asteiker
Copy link
Member

@asteiker asteiker commented Aug 1, 2023

Updated "Data Strategies for Future Us" in Readme link to point to Quarto rendered presentation. Otherwise I'm not sure where to locate the rendered slides from the repo currently.

Updated Data Strategies for Future Us link to point to Quarto rendered presentation.
@MattF-NSIDC
Copy link
Member

MattF-NSIDC commented Aug 1, 2023

Another way to expose a related website is with "About" settings (cog icon in screenshot):

image

You can specify a website (in this case "Use your GitHub Pages site" does the trick!)

image

I just set this up :) This doesn't take the user directly to the slides, though, so may be better to change "About" URL to exactly point to the slides or update the Quarto settings to put the slides at the "index" location.

@asteiker
Copy link
Member Author

asteiker commented Aug 1, 2023

Thanks @MattF-NSIDC ! I actually thought of updating the About section first but I wasn't actually sure if this update would be reflected in the PR (if I were to update this in my fork). If there's value in retaining the original link to the HTML in the Readme then we can revert back and keep the link in the "About".

@MattF-NSIDC
Copy link
Member

Ah yeah the "About" settings wouldn't be reflected in any branch I guess, so makes sense they wouldn't show up in a PR!

I think your change in this PR is probably also still a good idea, though! Just wanted to share that feature since I didn't realize until recently you could have a link in the "About" :)

@asteiker
Copy link
Member Author

@MattF-NSIDC or @andypbarrett Are you able to update the link under "About" from https://nsidc.github.io/data_strategies_for_future_us/ to https://nsidc.github.io/data_strategies_for_future_us/data_strategies_slides#/title-slide? The former link just takes you to a rendered version of the readme but I think we want this to take us to the slide deck instead. I don't have permission to make that update myself. Thank you!

@MattF-NSIDC
Copy link
Member

MattF-NSIDC commented Sep 13, 2023

Done!

EDIT: I think I gave you the correct permissions to edit the "About" section as well.

@asteiker
Copy link
Member Author

Thanks! And yes it looks like I can edit that section now. 🙂

@MattF-NSIDC
Copy link
Member

@asteiker I believe you now also have the ability to merge this PR if you'd like :)

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

2 participants