Skip to content
This repository has been archived by the owner on Feb 5, 2022. It is now read-only.

TranslocationData file needs redirect #8

Open
N3llz opened this issue Sep 28, 2016 · 12 comments
Open

TranslocationData file needs redirect #8

N3llz opened this issue Sep 28, 2016 · 12 comments

Comments

@N3llz
Copy link
Member

N3llz commented Sep 28, 2016

http://cellprofiler.org/linked_files/TranslocationData.zip needs a redirect (to http://d1zymp9ayga15t.cloudfront.net/TranslocationActivity/TranslocationData.zip)

@N3llz
Copy link
Member Author

N3llz commented Sep 28, 2016

original convo:

Jeanelle Ackerman 17 minutes ago
Regarding the redirect, the only place where the Translocation.zip file seems to exist on the website is on the /outreach page. I fixed that link. Is there a reason/situation where people would be accessing it a different way?
In the old website script, this path is wrong (that file isn't in /linked_files)
So 1) This makes me wonder how long it's been like that and if that pathway/url is irrelevant now. 2) Do you think it's ok as long as the new link works?
You can see it by clicking the .zip in the "Image-based screening for quantifying a translocation assay written exercise" Materials section on the outreach page.

Anne Carpenter 15 minutes ago
Indeed, the problem is that the link exists in the outside world. Whenever we make a major change to the website, we want to assess any URLs that existed in the old and create redirects to something sensible in the new site, where possible. Does that make sense?

Anne Carpenter 14 minutes ago
I don't actually know how to quickly figure out which URLs used to exist for an old site vs new.

@N3llz
Copy link
Member Author

N3llz commented Sep 29, 2016

Unfortunately, we need to use Jekyll before we can use redirects. I'll create an issue.

@N3llz N3llz closed this as completed Oct 7, 2016
@N3llz N3llz reopened this Oct 11, 2016
@N3llz
Copy link
Member Author

N3llz commented Oct 11, 2016

@mcquin Anne informed me this still does need to be addressed because although the link wasn't in the old website, it still exists from tutorials, so I reopened the issue

@mcquin
Copy link
Contributor

mcquin commented Oct 11, 2016

The only thing we can do about this is update the links in the tutorials. OR move the file off CloudFront and back onto cellprofiler.org.

@N3llz
Copy link
Member Author

N3llz commented Oct 11, 2016

@AnneCarpenter will it be sufficient to update the links in the tutorials? See claire's comment above.

@AnneCarpenter
Copy link
Member

Sadly, they exist out there in the universe so we don't have any control over it. Sounds like adding the file to cellprofiler.org instead of CloudFront is a reasonable solution unless it's painful in some way @mcquin ?

@mcquin
Copy link
Contributor

mcquin commented Oct 11, 2016

GitHub repositories are encouraged to not exceed 1GB in size. The current size of the website is ~0.014GB. The size of the file is ~0.010GB. I think we should be fine to host this.

@AnneCarpenter do you happen to know if there are any other similar files missing?

@AnneCarpenter
Copy link
Member

I see. Indeed, it sounds not too harmful to place this file in Github - but you're right to question whether it's one of many before doing so!

I wish I knew if there were any others missing. I'd hoped we'd have a listing of all files that used to reside at cellprofiler.org in order to check for any others. @N3llz is that going to be possible?

@N3llz
Copy link
Member Author

N3llz commented Oct 12, 2016

Well, what's odd is that this file wasn't residing in the old website - at least not in that path - so that's where it gets tricky because I wouldn't know where to even search for it, other than the tutorials you've mentioned? I'm not sure how we would even go about searching through tutorials for mysterious links like this one. @AnneCarpenter

@AnneCarpenter
Copy link
Member

Ah! Well, that explains how it got missed in your process then! I have a copy, will transfer.

@N3llz
Copy link
Member Author

N3llz commented Oct 12, 2016

@AnneCarpenter I have the file, but the path "http://cellprofiler.org/linked_files/TranslocationData.zip" will not work if people are trying to use that from some time ago (must have been even before the last version of the website). And like Claire said, we don't know if there are more like this (that would have also been missed in the transfer to the new site). @mcquin we should be fine to just change the link for this file to direct people to its Github location once we host it correct? is that what you're thinking?

@mcquin
Copy link
Contributor

mcquin commented Oct 14, 2016

@AnneCarpenter The link has been broken for a long time (since February 9 2016, or before). Either nobody else has noticed or they're silent or they came back here and got the correct link. I'm not sure if restoring this link is essential, though we can do it.

@N3llz I suggest leaving the CloudFront link for this file as it, so new users are taken to the correct location. We can place the file also at http://cellprofiler.org/linked_files/TranslocationData.zip so that anyone trying to access that link directly can get the file.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants