-
Notifications
You must be signed in to change notification settings - Fork 9
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
Database schemas added for user data, quests and settings #74
Conversation
ChatGPT4 assistance used to find out how to build schemas for Mongo |
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.
Pull Request Review:
-
Readability: Each part of the schema is well defined and commented with easy to understand names.
-
Code Quality: Code is of high quality and easy to read.
-
Testing: Unsure if the code can even be tested.
-
Functionality: Added database information storage required for the application.
-
Housekeeping: Everything looks clean and in order.
Additional Comments: Cannot open chatGPT message log will need others conformation on that.
linked fixed, thank you |
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.
Pull Request Review:
Readability: Code is formatted well with the appropriate comments. Easy to read!
Code Quality: High quality code, everything is easy to follow.
Testing: Don't think tests are needed? Other input needed.
Functionality: Great functionality added!
Housekeeping: Comments in all appropriate and necessary places, easy to read, easy to follow workflow. Looks good to me!
Additional comments: ChatGPT link works!
Overview
Please provide a brief overview of what this pull request aims to achieve. For instance, you might be fixing a bug, adding a new feature, or updating documentation.
Related Issue(s)
Closes issue #73 Missing MongoDB schemas
Changes
Users
Users
Quests
Settings
Added
Changed
Checklist before Merging
Additional Comments
Game data not included!