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.

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

Task#3 As a talent I want to estimate proofs of other talents by givi… #96

Merged
merged 2 commits into from
Apr 19, 2023

Conversation

Denis973
Copy link

…ng "kudos"

@Denis973 Denis973 added 🐸 backend Tasks for backend team user story #1 Tasks for 1st user story labels Apr 19, 2023
@Denis973 Denis973 added this to the Sprint 3.1 milestone Apr 19, 2023
@Denis973 Denis973 self-assigned this Apr 19, 2023
Copy link

@LordRenDS LordRenDS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice work

Copy link

@Maslyna Maslyna left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not ready yet.

Comment on lines 47 to 48
throw new ResponseStatusException(HttpStatus.FORBIDDEN,
"Talent can give only one “kudos“ for one proof");
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In this point you should use 409 status code CONFLICT

@Denis973 Denis973 merged commit bc51161 into S3.1US1 Apr 19, 2023
@Denis973 Denis973 deleted the Task#3 branch April 19, 2023 14:27
Maslyna added a commit that referenced this pull request Apr 19, 2023
* Refactor:
* remove Eager it Talent entity

* Task#1-2 (#93)

* Created Cudos entity and did some DB refactor
* Created 'GET(/proofs/{proof-id}/kudos)' endpoint in KudosController
* Created method 'getAmountKudosProof' in KudosService
* Created KudosRepository and add 'countByProof_Id' method

* Task#3 As a talent I want to estimate proofs of other talents by givi… (#96)

* Task#3 As a talent I want to estimate proofs of other talents by giving "kudos"

* bug fix

* S3.1 us1.task 4 (#97)

* Added don`t working service

* @manytoone relationSHIT

* Code refactor and bugfix

* Transactional

---------

Co-authored-by: Ren <[email protected]>
Co-authored-by: Ren <[email protected]>
Co-authored-by: Denis973 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐸 backend Tasks for backend team user story #1 Tasks for 1st user story
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants