Allow to enable/disable email signin via env variable
close #911
@yk I guess NEXT_PUBLIC_ENABLE_EMAIL_SIGNIN also needs to be added to the github-worklfow deploy-to-node.yaml & ansible ?
do we need to add the variable to https://github.com/LAION-AI/Open-Assistant/blob/main/ansible/deploy-to-node.yaml ?
So do I need to update other config? Also, should I rename it NEXT_PUBLIC_DISABLE_EMAIL_SIGNIN to allow incremental migration?
do we need to add the variable to https://github.com/LAION-AI/Open-Assistant/blob/main/ansible/deploy-to-node.yaml ?
Yes and in https://github.com/LAION-AI/Open-Assistant/blob/main/.github/workflows/deploy-to-node.yaml when we want to control the variable via github. As far as I understand the check for the variable defaults to false .. i.e. the feature is disabled if the variable is missing. This means we need to make sure it is properly set for staging and production .. otherwise email-signup will not work.
Actually wait, I'm going to un-approve this until I go through the sign up process and set the env variables. Merging this too quickly could break things.