Skip to content

Update Purdue details #1677

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

Merged
merged 3 commits into from
Mar 12, 2025
Merged

Update Purdue details #1677

merged 3 commits into from
Mar 12, 2025

Conversation

glentner
Copy link
Contributor

@glentner glentner commented Mar 6, 2025

Description

Just updating the name of our center and affiliate link.

Checklist:

  • [X ] I have previewed changes locally or with CircleCI (runs when PR is created)
  • [X ] I have completed any content reviews, such as getting input from relevant working groups. If no, please note this and wait to post the PR to the #website channel until the content has been settled.

When you are ready for a technical review/merge, post the for the link for the PR in the US-RSE Slack (#website) to ask for reviewers.

Copy link
Contributor

@crd477 crd477 left a comment

Choose a reason for hiding this comment

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

LGTM.

For the record, this was prompted by a transient urlcheck failure encountered as part of #1676.

@mrmundt
Copy link
Contributor

mrmundt commented Mar 7, 2025

@glentner - The URL you added is causing a failure; when I click on it, I get to this page:
Purdue website cert failure message

@glentner
Copy link
Contributor Author

glentner commented Mar 7, 2025

The web is hard. 😢

This happens in my own browser sometimes and not other times and I don't get it. Let me speak with our web team to see if I can't understand. It's the correct URL.

@glentner
Copy link
Contributor Author

glentner commented Mar 7, 2025

I think something is wrong with the intended automatic redirection to www.rcac.purdue.edu from rcac.purdue.edu.

@crd477
Copy link
Contributor

crd477 commented Mar 7, 2025

It redirected correctly for me in Firefox. If there are multiple servers involved, perhaps the redirect is only misconfigured on a subset.

@glentner
Copy link
Contributor Author

glentner commented Mar 7, 2025

We're investigating but in the meaning I'll probably just update the PR to include the www.

@glentner
Copy link
Contributor Author

glentner commented Mar 7, 2025

I've pushed a new commit that adds the explicit www which should avoid any issues.

@mrmundt mrmundt merged commit 375bc39 into USRSE:main Mar 12, 2025
2 of 3 checks 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.

None yet

3 participants