search

nsidc / usaon-benefit-tool

Application for configuring USAON Benefit Tool value tree analysis surveys and gathering input from respondents
https://usaon-benefit-tool.readthedocs.io/
MIT License
0 stars 3 forks source link

Legacy App issue #315

Open hazelshapiro opened 2 months ago

hazelshapiro commented 2 months ago

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.

mfisher87 commented 2 months ago

@hazelshapiro the sheet is published, correct?

hazelshapiro commented 2 months ago

Yes, this is the published url https://docs.google.com/spreadsheets/d/1envoTaQE59sGkgnZJU8L6KMICrjHZAOGsh6f998JGbI/pubhtml

mfisher87 commented 2 months ago

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.

rmarow commented 2 months ago

@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.