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.

Sign up for GitHub

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

Jump to bottom

Logger #51

Open
wants to merge 37 commits into
base: dev
Choose a base branch
from
Open

Logger #51

wants to merge 37 commits into from

Conversation

vswalker96
Copy link

Also create a Aspect class and configured so we can start logging
properly.
Pom file has updated dependency for postgres.

victoriawalker and others added 30 commits April 7, 2020 16:04
Fixed Update car information form so it submits
3d8a83d 
fixed updateUserInfo method to take in current userId.
Fixed all controller tests.
2e59080
Made users also return users car info
928adaf
Fixed Tests to use optional as was expected.
9ba9c88
@viccarogg
Merge pull request #2 from Batch-729-Jasdhir-Singh/Driver-Seats-Feature
9188c77 
Driver seats feature
@BVaihinger
new methods for recommending - doesn't fully return
b2cbc51
Refactoring of making the user
93265d3 
Refactored user creation and the corresponding tests on the user.
Merge remote-tracking branch 'origin/dev' into unitTestFix
8fd42a6 
Updated mine with master
Added Unique Constraint to Username.
6e08f11 
Also create a Aspect class and configured so we can start logging
properly.
Pom file has updated dependency for postgres.
Fixed Shortcuting Errors
7459bb8
@viccarogg
Merge pull request #3 from Batch-729-Jasdhir-Singh/unitTestFix
66001e9 
Unit test fix
@BVaihinger
Catches null values, appears functional, needs testing
9553a55
@viccarogg
updated H2 seeding script
a8d33db
@viccarogg
Merge branch 'dev' of https://github.com/Batch-729-Jasdhir-Singh/ride…
97d68ea 
…share-user-service into dev
Fix for passing a driver in an illegal state
f84d411 
W/ Testing!
@viccarogg
Merge pull request #4 from Batch-729-Jasdhir-Singh/driverfix
fbeea50 
Fix for passing a driver in an illegal state
@viccarogg
updated Batch to contain the office location. Updated data.sql to use…
7eb439d 
… batch locations and have users with home locations near their work locations.
@viccarogg
Merge pull request #5 from Batch-729-Jasdhir-Singh/data-script
d00fc5f 
updated Batch to contain the office location. Updated data.sql to use…
@viccarogg
fixed issue with column labeling in data.sql
f653ef5
@viccarogg
Merge pull request #6 from Batch-729-Jasdhir-Singh/data-script
7c56b12 
fixed issue with column labeling in data.sql
@BVaihinger
updated controller name, divided big method into helpers
4ed60e8
Merge branch 'dev' into Logger
2f03f8a
Fixed broken tests
b2aa7b2 
Fixed a couple broken tests after pojos where updated.
Refactored error sending for increased readability and added more rob…
0af1d8f 
…ust tests to it.
@viccarogg
Merge pull request #7 from Batch-729-Jasdhir-Singh/Utest
62ec39a 
Utest
@BVaihinger
commented out print statements, checks if user is rider, uses full ad…
bd65e5a 
…dresses
Aspect logging sends logs to file.
e755d2e
Logged HTTP responses.
07c8af5
@vswalker96
Merge pull request #8 from Batch-729-Jasdhir-Singh/Logger
0c6967e 
Logger
@BVaihinger
Added JUnit tests, commented out legacy code for testability
c8c1736
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@vswalker96 @viccarogg @BVaihinger @RobertGYoung