feat: add stand and get game features to blackjack game with animations #105

Open
jleibl wants to merge 5 commits from task/CAS-50/add_rest_blackjack_logic_with_frontend_animations into main
Member
No description provided.
jleibl added 3 commits 2025-03-27 14:45:35 +00:00
feat(blackjack): add action indicators and loading states
Some checks failed
CI / Get Changed Files (pull_request) Successful in 7s
CI / prettier (pull_request) Failing after 1m0s
CI / eslint (pull_request) Failing after 1m3s
CI / test-build (pull_request) Successful in 1m18s
CI / Checkstyle Main (pull_request) Successful in 1m31s
adc5bbd345
jleibl added 1 commit 2025-03-27 14:47:25 +00:00
style(blackjack): format code for better readability
Some checks failed
CI / Get Changed Files (pull_request) Successful in 6s
CI / eslint (pull_request) Failing after 8s
CI / prettier (pull_request) Successful in 20s
CI / Checkstyle Main (pull_request) Successful in 31s
CI / test-build (pull_request) Successful in 46s
23888ceb27
jleibl added 1 commit 2025-03-27 14:50:38 +00:00
refactor(blackjack): remove unnecessary comments and clean code
All checks were successful
CI / Get Changed Files (pull_request) Successful in 6s
CI / prettier (pull_request) Successful in 19s
CI / Checkstyle Main (pull_request) Successful in 27s
CI / eslint (pull_request) Successful in 45s
CI / test-build (pull_request) Successful in 56s
05322e3d83
jleibl changed title from task/CAS-50/add_rest_blackjack_logic_with_frontend_animations to feat: add stand and get game features to blackjack game with animations 2025-03-27 14:50:56 +00:00
jank requested changes 2025-03-27 17:24:16 +00:00
@ -26,1 +26,4 @@
@Version
@JsonIgnore
private Long version;
Owner

what is this version even used for?

what is this version even used for?
@ -21,1 +21,4 @@
@Version
@JsonIgnore
private Long version;
Owner

why version here as well

why version here as well
@ -49,11 +61,14 @@ export default class BlackjackComponent {
this.currentGameId.set(game.id);
this.currentBet.set(game.bet);
this.gameInProgress.set(game.state === 'IN_PROGRESS');
Owner

create enum for game states

create enum for game states
@ -35,1 +87,4 @@
@Output() doubleDown = new EventEmitter<void>();
@Output() leave = new EventEmitter<void>();
calculateHandValue(cards: Card[]): number {
Owner

extract this into a service and all the other methods

extract this into a service and all the other methods
@ -0,0 +1 @@
/* No custom styles needed */
Owner

just remove this file

just remove this file
@ -0,0 +1,22 @@
import { ComponentFixture, TestBed } from '@angular/core/testing';
Owner

remove this file its unnecisary

remove this file its unnecisary
All checks were successful
CI / Get Changed Files (pull_request) Successful in 6s
CI / prettier (pull_request) Successful in 19s
Required
Details
CI / Checkstyle Main (pull_request) Successful in 27s
Required
Details
CI / eslint (pull_request) Successful in 45s
Required
Details
CI / test-build (pull_request) Successful in 56s
Required
Details
This pull request has changes conflicting with the target branch.
  • backend/src/main/java/de/szut/casino/blackjack/BlackJackGameController.java
  • backend/src/main/java/de/szut/casino/blackjack/BlackJackService.java
  • backend/src/main/java/de/szut/casino/blackjack/BlackJackState.java
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin task/CAS-50/add_rest_blackjack_logic_with_frontend_animations:task/CAS-50/add_rest_blackjack_logic_with_frontend_animations
git checkout task/CAS-50/add_rest_blackjack_logic_with_frontend_animations
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: SZUT/casino#105
No description provided.