Open
Conversation
Update README.md and add local.env.example **Why** We want to have a nice example of environment variables that can be used when developing locally. **How** Update README.md to call out the importance of having a local.env as well as add an example env file.
…and Teams, and Leagues getting Teams. (#26)
…User Emails, and Create tests for the Roster model POST and GET. (#28) * Begin changing Roster tracking of Users by ID instead of Email. * Begin adding Roster tests. * Roster testing copying goals test * Added Users to the test to build a Roster * Added Users to the test to build a Roster(2) * Fixed test for /backend/internal/db/roster.go
- Split PenaltyType into its own file - Created endpoints for: - Get Penalties by ID - Update Penalty - Delete Penalty - Get Penalties by Game - Get Penalties by Team - Get Penalties by Player - Create Penalty Type - Update Penalty Type - Delete Penalty Type - Fixed `DeletedAt` in base `DbModel` to allow for soft delete jak103#18 jak103#178 Note: Splitting unit tests into a separate PR, this is getting too big
* Added docs for auth in README * Added password hashing and saving into DB for users * Add seeder for users * Updated users docs endpoint * Add more commands in makefile for backend
* Working on possible solutions to the bug. Found three possibilities i should discuss with team.(pointers rather than actual values, using sql.NullDataType, and gopkg.in/guregu/null.v3 package which seems promising as it can be unmarshalled easily.) * Have a working go.validate solution as well as a working gorm not null solution working. I need feedback on which to move forward with, as both are significant changes. * got go validate working for penalties, and changed all penalty fields to be required. * Added "required" validation for goals * Working on venues validation, but now docs aren't working for it. * Working on bug in the games endpoint. * added validation for leagues and games. * Added validation to rosters. * got validation working for seasons * added validation to teams. Also changed the team field in game to use a pointer so that it didn't auto fail the recursive validation. --------- Co-authored-by: tannerwheeler <tannerleonwheeler@gmail.com>
* added bare bones season database test * added files for seasons api endpoint testing * added seasons api tests * removing to see if it will pass --------- Co-authored-by: Dingleberryipod <a02330716@usu.edu>
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
6 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.
No description provided.