-
Notifications
You must be signed in to change notification settings - Fork 1
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
Idan domain ucc03 0 #24
Merged
Merged
Changes from all commits
Commits
Show all changes
50 commits
Select commit
Hold shift + click to select a range
9ede6f5
start working on the tests of game center
idanizi a5f0916
start working on the tests of game center
idanizi 421ce80
leagues level, enter a room. create a room
idanizi 004f14b
room creation
idanizi bdcdfc9
adding 2 exceptionsworking on gamecenter, game config min/max enforce…
idanizi ef7fc46
join next hand
idanizi 9b45063
wallet:
idanizi 7114134
uc013 stand up from table
idanizi 406e5ec
working on user exposing methods of gamecenter. todo: make the gamece…
idanizi 780b9a5
game center is now a singleton. logger is thread safe (tests pass :) )
idanizi 4d402ff
user.enter room
idanizi 5cbaf57
player test - pass
idanizi 6547ec0
player test - pass
idanizi 62ba035
service layer example
idanizi 3d93394
renaming NewRoom request / result to CreateNewRoom...
idanizi 55087d1
service layer - to add - user not found exception
idanizi 20017a4
working on the service and communication layer - ucc03 room service(s…
idanizi a56b784
clearing some uneeded files at Poker.API, todo: need to clear more. m…
idanizi c6fd335
initial room controller, todo excemption catching
idanizi a434bec
room service and controller
idanizi a9c149a
setting Newtonsoft.Json as the json formatter (and xml formatter remo…
idanizi 49cd55f
testing room
idanizi 3b9f3ef
editing todos, testing room and bug fixing
idanizi dd99c53
fixing game config
idanizi eed1722
room(:player) ctor test pass
idanizi a188238
room(:player, :preferences) ctor test pass
idanizi 049640b
working on room(:player, :gameconfig) ctor test
idanizi 0ce05e4
working on room(:player, :gameconfig) ctor test
idanizi bf1f9db
room(:player, :gameconfig) ctor test pass
idanizi e33952e
join player to table - room - pass
idanizi 06c3e7c
create player - room - pass
idanizi 888637a
take and leave chair - room - pass
idanizi c45618d
enter room & create new room - gamecenter - pass
idanizi 6fc5e29
working on join next hand tests
idanizi 7475af0
working on find rooms
idanizi 717aaac
working on find rooms
idanizi eff02ed
working on find rooms
idanizi 017e206
find room by criteria - gamecenter - level, palyer - pass
idanizi 12edd8f
gamecenter test all pass. working on user tests of ucc03 methods - la…
idanizi 053b018
gamecenter test all pass. working on user tests of ucc03 methods - la…
idanizi b34ea9e
all tests pass ucc03
idanizi e2258b7
fixing for ci
idanizi e71afa6
fixing for ci
idanizi 18b0732
fixing for ci
idanizi be9f021
fixing for ci
idanizi 83b1fd4
Merge remote-tracking branch 'origin/master' into idan_domain_ucc03_0
idanizi 29c6e5d
fixing build bugs after merge. todo - paly move test not fail - fixme
idanizi 218bb3a
all tests pass
idanizi 5193a2d
fix for PR comments
idanizi fe633eb
fix for PR comments - all test pass
idanizi File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
25 changes: 0 additions & 25 deletions
25
Poker.BE/Poker.BE.API.Tests/Controllers/HomeControllerTest.cs
This file was deleted.
Oops, something went wrong.
60 changes: 0 additions & 60 deletions
60
Poker.BE/Poker.BE.API.Tests/Controllers/ValuesControllerTest.cs
This file was deleted.
Oops, something went wrong.
This file contains 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
This file contains 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
This file was deleted.
Oops, something went wrong.
This file contains 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
This file was deleted.
Oops, something went wrong.
This file contains 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
This file contains 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
This file contains 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
Oops, something went wrong.
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.
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.
new Service.Services? seems like bad smell or maybe I miss something
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.
you miss something.