change /config/config.js to config/config.js

NCEAS / metacatui

MetacatUI: A client-side web interface for DataONE data repositories

https://nceas.github.io/metacatui

Apache License 2.0

41 stars 27 forks source link

change /config/config.js to config/config.js #2463

Closed artntek closed 2 months ago

artntek commented 3 months ago

see #2454 - Change appConfigPath in index.html to the relative path config/config.js

NOTE: BEFORE MERGING

...we should ensure that this change will be compatible with all existing (non Kubernetes) installations.

robyngit commented 3 months ago

[x] Updated our current upgrade script to match and replace /config/config.js OR config/config.js (https://github.nceas.ucsb.edu/NCEAS/metacatui-config/issues/1)

artntek commented 3 months ago

For testing, I created k8s deployments for knb, arctic, and cerp, all using the relative path in this feature-2454-relative-appConfigPath branch. What additional verification do you recommend before merging, @robyngit?

Robyn replied in slack 7/8/24:

I tested it on the MetacatUI side, I think it'll be ok and we can merge it