-
Notifications
You must be signed in to change notification settings - Fork 27
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update README #149
Comments
Hmm. To me, setting FCM based notification is not yet production ready, so I don't want to include it in readme. We might remove it in the future if it didn't work for us. |
Sure. Switching to false should work. I guess devs could manually set it to true when they work with the SMS service and use MockSMS at all other times. |
I don't know how it is being used now, but the original idea was to have |
The default profile being used for the application runtime is local. The README also does not mention setting the profile at runtime to anything else. However, local has
sms.enabled=true
but does not mention how a developer can set up their credentials for the same. This leads to an error while running sayingFCM Credentials are not found
. The Dockerfile specifies the usage of thedev
profile for running the application. I think the README should be updated with the same or mention how a user can set up the credentials.The text was updated successfully, but these errors were encountered: