Skip to content

feat: Routing from /storage#575

Open
hetd54 wants to merge 2 commits intomainfrom
routing-storage
Open

feat: Routing from /storage#575
hetd54 wants to merge 2 commits intomainfrom
routing-storage

Conversation

@hetd54
Copy link
Copy Markdown
Collaborator

@hetd54 hetd54 commented May 5, 2026

There was some confusion about the url change from the old site:

I just found https://ccv.brown.edu/storage (old website) has been moved to https://ccv.brown.edu/services/file-storage-and-transfer?

Thoughts on this redirect? With this PR, folks who visit the old storage page will get redirected to the new page instead of a 404

Per @yangliu2009 :

Thank you for looking into the issue, Heather. I’d actually recommend not setting up redirects at all. Instead, we can just inform users that the CCV website has migrated and provide the new URLs. This way, we avoid the technical debt of maintaining the old URLs indefinitely. Just my 2c.

@hetd54 hetd54 requested review from eldu and paulstey May 5, 2026 17:10
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 5, 2026

App Preview

(58fed2c)

@paulstey
Copy link
Copy Markdown
Member

paulstey commented May 5, 2026

I'd be inclined to agree with Yang, and I would vote not to add redirects. That storage page is reachable within two clicks—technically, it's a hover and a click. It's not hard to find!

I'm curious what others think about this.

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