-
Notifications
You must be signed in to change notification settings - Fork 3
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
Legacy App issue #315
Comments
@hazelshapiro the sheet is published, correct? |
Yes, this is the published url https://docs.google.com/spreadsheets/d/1envoTaQE59sGkgnZJU8L6KMICrjHZAOGsh6f998JGbI/pubhtml |
Thanks! It's not the first time they've done this. How inconsiderate! We should be saving the full URL in our index sheet, and updating the code to use that full URL instead of templating based on the old URL structure. The change sounds minor, but it's been some time since we deployed this app, and we may have taken shortcuts because we only expected to deploy it once, so there may be additional minor challenges there. What do you think @rmarow ? We have that deployment set up in this private repo, but I don't remember if you worked on it or not: https://github.com/nsidc/usaon-vta-webapp-legacy EDIT: We're still using CircleCI on that repo! Long time since I've used that. |
@MattF-NSIDC I don't recall doing any deploying of the legacy app during my time on this project besides reviewing it for work on the new app. Very frustrating that google changes the URL! I do agree with you that deploying again could provide some minor challenges. |
Good to fix/address this if it's quick since we're still using the legacy app for demonstration purposes.
Index of Datasets
README doc
New spreadsheet I am trying to add
It looks to me like Google changed the format of it's links - I can't get the new spreadsheet ID in a format where it starts with 2PACX also where it used to say "/spreadsheets/d/e/" the URL is now just "/spreadsheets/d/"
I think that's why I am getting a "No Links" error.
The text was updated successfully, but these errors were encountered: