You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Not something I'd halt merge for, but do we need a consistent way to expose API Roots so that they can be used externally / overridden by a consuming app?
Related to #26
I thought about this as I was building the NWPS hooks. I think it makes sense for APIs that we control (e.g. CDA) for development purposes, but I think it would be rare to have a situation in which we would be using different URLs for the same external API. Occasionally the URL will need to be updated, such as when NWPS went from dev to prod, but that's a change that would theoretically be required across-the-board for anyone using groundwork.
Activity
jbkolze commentedon May 3, 2024
Related to #26
I thought about this as I was building the NWPS hooks. I think it makes sense for APIs that we control (e.g. CDA) for development purposes, but I think it would be rare to have a situation in which we would be using different URLs for the same external API. Occasionally the URL will need to be updated, such as when NWPS went from dev to prod, but that's a change that would theoretically be required across-the-board for anyone using groundwork.
willbreitkreutz commentedon May 3, 2024
Yeah, my only thought was a single place for them all, if they update regularly or not...