LD4P / sinopia_acl

node.js based code to interact with WebACL data on sinopia server
Apache License 2.0
0 stars 0 forks source link

ENV vars and ??? in AWS setup for sinopia_acl #65

Closed ndushay closed 5 years ago

ndushay commented 5 years ago

I believe we will need these, obviously configured per appropriate environment (dev, stage, prod):

(for reference: ENV vars for editor container: LD4P/sinopia_editor#458)

ndushay commented 5 years ago

tagging @mjgiarlo so he can chime in with any stuff he is aware of.

mjgiarlo commented 5 years ago

See also https://github.com/LD4P/sinopia_editor/pull/462

ndushay commented 5 years ago

These vars are now "definite", per PR #66.

rsmith11 commented 5 years ago

I don't know what the password is for the sinopia_client-dev-tester user in Cognito. If somebody could slack that to me that would be great.

mjgiarlo commented 5 years ago

@rsmith11 :speech_balloon:

I don't know what the password is for the sinopia_client-dev-tester user in Cognito. If somebody could slack that to me that would be great.

https://github.com/sul-dlss/shared_configs/blob/sinopia-dev/README.md

mjgiarlo commented 5 years ago

Here is an unresolved ENV var question that is currently blocking a PR

It is now resolved, and the text above is accurate.

mjgiarlo commented 5 years ago

@rsmith11 @ndushay @jermnelson Do we think this is done? I'm wondering if we should close this issue and, if changes are needed as we discover bugs, ticket those separately.

mjgiarlo commented 5 years ago

@rsmith11 What say you about :point_up: ? If you concur, feel free to close the issue.