damondouglas commented on a change in pull request #15956: URL: https://github.com/apache/beam/pull/15956#discussion_r749463537
########## File path: playground/frontend/lib/constants/api.dart ########## @@ -16,4 +16,5 @@ * limitations under the License. */ -const String kApiClientURL = 'https://datatokenization.uc.r.appspot.com'; +const String kApiClientURL = + 'https://backend-dot-datatokenization.uc.r.appspot.com'; Review comment: May we consider not hardcoding the URL in the codebase? Apologies that I missed it in an earlier review. One possible way to solve it is using a `config.json` file like [this](https://github.com/damondouglas/fs-auth-example/tree/main/frontend/assets#configjson) and [this](https://github.com/damondouglas/fs-auth-example/blob/main/frontend/lib/config.dart). The friction, however, is that it requires creating and maintaining this file. Perhaps we can add a gradle task that creates this config, acquiring the endpoint URL from a gradle.properties file in `playground/frontend/gradle.properties`. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
