Skip to content
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/py-upgrade: Clean up scripts, configurations, and fix README #36

Merged
merged 9 commits into from
Jul 14, 2024

Conversation

Veivel
Copy link
Contributor

@Veivel Veivel commented Jul 14, 2024

Summary

  • Fixed app configs, deployment configs and configuration guides
  • Renamed some files, moved other files
  • Rewrote README more up-to-date, with a link to the old README if still necessary
  • [IMPORTANT] Database initialization is now done manually.

About DB Initialization

Previously, a custom docker image was used to load a .sh script to the MongoDB docker entrypoint. This would create a MongoDB user with the credentials that the server would eventually use to connect. After a short discussion we decided to make this manual, under the pretense that database initialization should be done manually as best practice.

Writing this to avoid tribal knowledge, for any future maintainer feel free to contact me about any db connection-related issues :)

@Veivel Veivel merged commit 28f69b8 into master Jul 14, 2024
@Veivel Veivel deleted the feat/py-upgrade branch July 14, 2024 17:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant