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

Stale logistics information #9

Closed
wants to merge 1 commit into from
Closed

Stale logistics information #9

wants to merge 1 commit into from

Conversation

naclomi
Copy link
Contributor

@naclomi naclomi commented Aug 5, 2024

One of the attendees pointed out that the dorm check-in dates on the logistics page were out of date. This PR attempts to update them based on the information here, though I might have missed some details and ultimately it might be a better idea to just remove the details from logistics.md and link directly to the Notion page. Would love to hear folks' thoughts.

@jomey
Copy link
Member

jomey commented Aug 5, 2024

+1 for removing any (if not the entire page) and point to the resource you linked to?

JupterBook can have external URLs as part of the TOC.

@JessicaS11
Copy link
Member

Thanks @naclomi (and thanks for the well-named branch so I saw this when I went to create one to update the logistics info)! We are on similar wavelengths: I opened a PR to add the logistics info to our event page, with corresponding GeoSMART and IS2 PRs to remove this page entirely from the Jupyterbooks. Happy to take a different path if there's preference. Mark recently reminded me he created two logistics pages and suggested we use the more generic one here (versus the OT-specific one linked above, which has some content on reimbursement that might confuse participants).

@naclomi
Copy link
Contributor Author

naclomi commented Aug 5, 2024

@JessicaS11 -- agreed that your PR's approach is preferable, thanks for rolling out the changes here and the other repos :) . I'll close this one.

@naclomi naclomi closed this Aug 5, 2024
@jomey jomey deleted the logistics-update branch September 12, 2024 16:58
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.

3 participants