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

Artscope 3.1.4 릴리즈 #164

Merged
merged 119 commits into from
Apr 23, 2024
Merged

Artscope 3.1.4 릴리즈 #164

merged 119 commits into from
Apr 23, 2024

Conversation

Hoon9901
Copy link
Contributor

No description provided.

Hoon9901 and others added 30 commits March 22, 2024 21:03
- MagazineCategory에 slug, parent_id 칼럼 추가
- MagazineCategory에 같은 parent_id를 둔 요소는 같은 이름을 쓸 수 없게 조건 생성
- get ("/slug") 슬러그를 통해 하위 카테고리를 찾을 수 있습니다
- delete ("id") 해당 카테고리를 삭제할수 있습니다
- put ("id") 해당 카테고리를 수정할수 있습니다
- 삭제시 해당되는 매거진이 있으면 삭제되지 않습니다.
- 생성시 이름이나 슬러가가 중복되면 에러코드를 반환합니다. (기존 : 에러가 그대로 노출)
- 랜덤 함수 시드를 동적(System.currentTimeMillis())을 사용하도록 변경
update(MagazineCategory) -> changeParentCategory(MagazineCategory)
- 슬러그 중복 카테고리 검증 로직을 변경했습니다
- 에러 메시지들을 구체화 했습니다
- update 시 올바르게 카테고리 조건들이 유효한지 검사하도록 했습니다
- 카테고리 최대 2단계 까지만 생성 가능하다 테스트의 에러 메시지 확인부분을 e2e에 맞게 변경했습니다.
- 테스트 코드를 추가하였습니다
- magazineCategory의 검증 코드를 추가했습니다
- magazineCategory의 생성 수정시 category를 검증하도록 수정헀습니다
- magazine 생성, 수정시 카테고리 슬러그를 받도록 변경했습니다.
- magazine생성시 categorySlug를 받도록 변경됨에 따라 수정했습니다.
- 해당 카테고리를 가진 매거진이 있을경우 에러를 검증하는 테스트코드 추가
- group, group_user 에 대한 entity 클래스를 작성했습니다.
- group_description에서 description으로 변경된 column이 누락된걸 추가했습니다.
haroya01 and others added 27 commits April 16, 2024 20:20
- member.getTeamUser()에서 null 체크를 isEmpty()로 변경
- 기존 GET /api/members/profile
- 이후 GET /api/auth/me
- 테스트 코드 수정
- 코드 포맷팅
- TeamUserRole을 반환할 수 있도록 추가
- 가장 오래된 순서대로 반환하도록 변경
- List<TeamResponse.ProfileGet>를 반환하도록 변경
-> 멤버가 참여하는 팀과 권한 조회
-> teams에서 members로 이동
- 보다 의미가 명확하게 메서드 명을 수정
…터-수정

Art 184 be 내가 속한 팀 정보 api 응답 데이터 수정
Copy link

github-actions bot commented Apr 23, 2024

Qodana Community for JVM

1 new problem were found

Inspection name Severity Problems
Mismatched query and update of collection 🔶 Warning 1

💡 Qodana analysis was run in the pull request mode: only the changed files were checked
☁️ View the detailed Qodana report

Contact Qodana team

Contact us at qodana-support@jetbrains.com

Copy link

codecov bot commented Apr 23, 2024

Codecov Report

Attention: Patch coverage is 91.66667% with 57 lines in your changes are missing coverage. Please review.

Project coverage is 72.06%. Comparing base (7187b87) to head (31d4db8).
Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff              @@
##               main     #164      +/-   ##
============================================
+ Coverage     70.32%   72.06%   +1.74%     
- Complexity     1600     1721     +121     
============================================
  Files           240      254      +14     
  Lines          5817     6380     +563     
  Branches        258      295      +37     
============================================
+ Hits           4091     4598     +507     
- Misses         1536     1575      +39     
- Partials        190      207      +17     
Files Coverage Δ
...om/example/codebase/controller/AuthController.java 100.00% <100.00%> (ø)
...odebase/controller/MagazineCategoryController.java 100.00% <100.00%> (ø)
.../example/codebase/controller/MemberController.java 71.95% <100.00%> (-2.75%) ⬇️
...om/example/codebase/controller/TeamController.java 100.00% <100.00%> (ø)
...xample/codebase/controller/TeamUserController.java 100.00% <100.00%> (ø)
.../codebase/domain/follow/service/FollowService.java 100.00% <100.00%> (+4.65%) ⬆️
...e/domain/magazine/dto/MagazineCategoryRequest.java 92.30% <100.00%> (+92.30%) ⬆️
...codebase/domain/magazine/dto/MagazineResponse.java 89.18% <100.00%> (+3.73%) ⬆️
...domain/magazine/repository/MagazineRepository.java 100.00% <100.00%> (ø)
...main/magazine/service/MagazineCategoryService.java 100.00% <100.00%> (ø)
... and 26 more

... and 1 file with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9fe90e5...31d4db8. Read the comment docs.

@Hoon9901 Hoon9901 merged commit 47b58cf into main Apr 23, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants