search

RMI-PACTA / app

API + web frontend for PACTA infrastructure
MIT License
2 stars 0 forks source link

Alex/workflow update json #202

Closed AlexAxthelm closed 1 month ago

AlexAxthelm commented 1 month ago

@bcspragu I was able to reproduce your findings, and thanks to @jdhoffa for spotting the mismatch between $YEAR in the script, and the values defined in $JSON_INPUT.

This PR moves defining that JSON string up into the same case as the other year-dependent envvars, so you should just be able to specify that now.

Tested on my machine

bcspragu commented 1 month ago

Ah, I saw @jdhoffa's message before I saw this PR and I fixed it in a slightly different way in https://github.com/RMI-PACTA/app/pull/201/commits/a3bff4d6cea27dd9657a213e3443f2d91d4e7e4a

I'm going to merge that one just because it's less duplicate-y, but thanks for the PoC! In any case, I'll throw this script away once I have the actual service integration working.