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 URLs for SP22 Semester #28

Merged
merged 3 commits into from
Apr 15, 2022
Merged

Update URLs for SP22 Semester #28

merged 3 commits into from
Apr 15, 2022

Conversation

ariporad
Copy link
Collaborator

This PR updates the inferred URLs for the SP22 semester.

ariporad and others added 3 commits April 15, 2022 14:50
Riccardo seems to have changed the format.
The new scheme prevents multiple urls ending in index.html from
clashing, and ensures an extension is always present.
@newsch
Copy link
Member

newsch commented Apr 15, 2022

Thanks for fixing this!
I rebased your commits off of the new master and updated the cached filenames to work with the new urls (and hopefully any future ones).
I also added you as a collaborator to the repo.

@newsch newsch merged commit c22a576 into olin:master Apr 15, 2022
@ariporad ariporad deleted the patch-1 branch April 16, 2022 20:03
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.

2 participants