-
Notifications
You must be signed in to change notification settings - Fork 1
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
feat/auth #49
base: dev
Are you sure you want to change the base?
Conversation
to run:
port: "8080"
db:
host: localhost
port: "3306"
name: rmx-dev-test
user: rmx
password: postgrespw
auth:
enable: true
google:
client_id: <CLIENT_ID>
client_secret: <CLIENT_SECRET>
cookie_key: <COOKIE_KEY>
dev: true
visit |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The structure looks fine to me. However, there are a few issues stemming from moving the migration files. We'll either need to move them back or update the sqlc config and test migration file paths.
Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change
Fixes # (issue)
Dependent # (issue)
Current status
Semantic Versioning
Please delete options that are not relevant. Ensure same has been covered in the commit message as well.
fix
changefeat
changeBREAKING CHANGE
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration
Automated
Manual
Checklist: