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 proceedings links for new proceedings site #61

Merged
merged 4 commits into from
Sep 6, 2024

Conversation

fwkoch
Copy link
Contributor

@fwkoch fwkoch commented Aug 30, 2024

This updates proceedings links across the site:

  • For links like https://conference.scipy.org/proceedings/scipy<year>, I updated to https://proceedings.scipy.org/<year>
  • For all the aws redirects on proceedings index and individual paper pages, I updated the redirects to https://doi.org/<doi> (which now point to proceedings.scipy.org)

While creating this PR I also confirmed that all the new links exist and correctly resolve to the new Curvenote proceedings site.

Note - I did not re-build the static site after these changes; this was mostly an automated find-and-replace job. Happy to make changes if there is something here that does not work with the static site build!

@rowanc1 rowanc1 requested a review from achabotl August 30, 2024 16:01
Copy link
Member

@achabotl achabotl left a comment

Choose a reason for hiding this comment

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

I think this should work. Thank you all for you hard work bringing the proceedings onto Curvenote and for not breaking the web. :D

@achabotl achabotl merged commit 6f362c8 into scipy-conference:main Sep 6, 2024
1 check passed
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