CornellSystemsEngineering/SYSEN-5160

Inconsistent links

Closed this issue · 7 comments

Hi,

Thanks for your submission. I noticed you posted links to two different repos in the README:

| ![](https://streamlit.io/images/brand/streamlit-mark-light.png) | A Smart Supply Chain Wizard | A great tool will help the company better manage the supply chain | [Supply Chain Wizard](https://medium.com/@dh734/supply-chain-wizard-89263ccd13c3) | [App](https://share.streamlit.io/fcelya/sysen5160-scm-tool/main/main.py) / [Repo](https://github.com/bellesjchen/sysen5160-scm-tool) | Sijie Chen [[M](mailto:chensijie1225@gmail.com)\|[L](https://www.linkedin.com/in/sijie-c-8559a6220/)] <br />Donghao Huang [[M](mailto:dh734@cornell.edu)\] <br />Fernando Celaya [[M](mailto:fernando.celaya@alu.comillas.edu)\|[L](https://www.linkedin.com/in/fernando-celaya-oyon/)]<br /> Keshan Chen [[M](mailto:kec180@cornell.edu)] |

Which one is the one marked for submission? Also, could you add instructions on how to upload a sample .csv file so I can take a closer look?

Best,

Patrick

Hi Sijie,

I don't see a link to the final repo in the google doc link you posted.

Could you please adapt edit the correct link in line 7, specifically this: [App](https://share.streamlit.io/fcelya/sysen5160-scm-tool/main/main.py) / [Repo](https://github.com/bellesjchen/sysen5160-scm-tool)

Thank you,

Patrick

Hi to both,

Both repos had the same working version of the application, but in order to be more consistent I have changed the Repo link so it points to the same one as the App link (bellesjchen/sysen5160-scm-tool currently is not uploaded to the Streamlit cloud so we pointed both to fcelya/sysen5160-scm-tool). The only difference between both repositories is the Readme, which we have changed in fcelya/sysen5160-scm-tool so it explains a little bit more about how to upload a custom .csv file.

If the readme needs to hold the whole report we can add that.

@fcelya @bellesjchen Thanks! One repo was 7 commits ahead of the other one, didn't mean to be pedantic just wasn't sure which one to look at. Readme doesn't need to hold the whole report, all good.

Thanks,

Patrick

No worries we understand the concern. The commits were just due to the differences in the readme.