Feature/133 persistence into database#152
Merged
Merged
Conversation
…nce-into-database
Remove annotation and use a normal constructor to reduce mental overhead when using the service class. With constructor and javadoc referencing how to build constructor becomes easier.
getFirst() results in compilation error, cannot find symbol error
Source Baeldung website (Forgot when creating class and threw null constraint violation when testing)
…rsistence-into-database
Forgot to use the new custom GameManager, turns out it was not needed? (Tests were failing due to invalid word so custom GameManager was added.)
Transactional should be auto configured since we are using SpringBoot if I understand correctly
Saving to DB is only meant to happen after a turn is over.
…rsistence-into-database
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Context
This PR contains the addition of persistency mapping functionality into our DB.
Description
The addition of these classes allow us to map the current game state, lobby and player to the entity classes. These will then be stored in the database.
Changes in the code base
Changes outside the code base
N/A
Additional information
N/A