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

[Feature] Newsnippet 서비스 구현 #33

Merged
merged 168 commits into from
Apr 17, 2024
Merged

[Feature] Newsnippet 서비스 구현 #33

merged 168 commits into from
Apr 17, 2024

Conversation

moomint8
Copy link
Member

반영 브랜치

dev -> main

Leegiyeon and others added 30 commits April 3, 2024 12:36
@Leegiyeon
build(gradle): dependency에 JPA 추가
6b0bfb1
@Leegiyeon
ci(gradle): ci 파일 추가
2c21a41
@Leegiyeon
build(gradle): dependency에 mariadb 추가
0cc4698
@Leegiyeon
build(gradle): dependency에 jar 설정 추가
82c9ad7
@Leegiyeon
ci(gradle): java distribution 수정
a40e408
@Leegiyeon
ci(gradle): gradle 설정 수정
8ebe9d6
@Leegiyeon
ci(gradle): gradle 설정 수정
6d37423
@Leegiyeon
ci(gradle): gradle 설정 수정
d88cd01
@Leegiyeon
ci(gradle): gradle 설정 수정
592c11f
@Leegiyeon
ci(gradle): gradle 설정 수정
ac650c6
@Leegiyeon
test(github action): 테스트용 코드 작성
e3c26c8
@Leegiyeon
ci(gradle): gradle 설정 수정
09851fe
@Leegiyeon
ci(gradle): gradle 설정 수정
7ee8846
@Leegiyeon
ci(gradle): gradle 설정 수정
ccc072f
@Leegiyeon
ci(gradle): gradle 설정 수정
f94e00b
@Leegiyeon
ci(gradle): gradle 설정 수정
ed12582
@Leegiyeon
ci(gradle): gradle 설정 수정
8bb6a6d
@Leegiyeon
ci(gradle): gradle 설정 수정
1d21aa1
@Leegiyeon
ci(gradle): gradle 설정 수정
3d38ab4
@Leegiyeon
ci(gradle): gradle 설정 수정
c2e7142
@Leegiyeon
ci(gradle): gradle 설정 수정
91114cd
@Leegiyeon
ci(gradle): gradle 설정 수정
86002a7
@Leegiyeon
ci(gradle): gradle 설정 수정
3d9ccc0
@moomint8
build(gradle): build.gradle dependency 추가
d9cdeb8 
JWT, OAuth, ModelMapper 라이브러리를 추가하고 각 라이브러리를 구분하기 쉽도록 주석이 추가되었습니다.
@moomint8
feat(Entity): 각 Entity 생성
cce0c1b 
DB에 맞는 엔티티를 생성했습니다.
엔티티는 DB와 같은 구조이므로 추후 개발의 편의성을 위해 미리 추가하게 되었습니다.
@moomint8
refactor(Entity): 각 Entity FK 수정
673d645 
JPA의 N+1 문제 예방을 위해 각 관계를 끊었습니다.
실제 DB에서는 관계가 맺어져 있으므로 부모 테이블을 먼저 조회한 뒤 id를 추출하여 다시 자식 테이블에 요청하는 방식으로 구현할 예정입니다.
추후 변경 시 다시 기재하겠습니다.
@Sosohy
feat(DTO) : QuizDTO 추가
19602d1
@Leegiyeon
ci(gradle): gradle 설정 수정 및 임시 테스트 코드 생성
669c59c
@Leegiyeon
ci(gradle): gradle 설정 수정 및 임시 테스트 코드 생성
11e7cec
@Leegiyeon
ci(gradle): gradle 설정 수정
2fbc43e
Leegiyeon and others added 20 commits April 12, 2024 17:53
@Leegiyeon
Merge pull request #27 from Triumers/quiz
acc4f67 
[Refactor] 정답률 계산 메소드 수정 및 사용하지 않는 메소드 삭제
@Leegiyeon
docs(Template): PR template 수정
c92a19b
@moomint8
refactor(CORS): Spring Security에서 CORS 처리 구현
25943a7 
SpringSecurity에서 CORS를 처리하도록 구현했습니다.
@moomint8
Merge pull request #28 from Triumers/user
8d26619 
[Refactor] CORS 처리 개선
@moomint8
refactor(DTO): DTO 계층 수정
b4da92f 
DTO의 계층을 더 적합하도록 수정했습니다.
(Domain -> Application)
@moomint8
test(Solved): 퀴즈 답안 제출 시 유저 정보 업데이트 테스트 구현
9288730 
퀴즈 답안 제출 시 정답, 오답 여부에 따라 유저 정보 업데이트를 테스트하는 메소드가 구현되었습니다.
@moomint8
feat(Solved): 퀴즈 답안 제출 시 유저 정보 업데이트 기능 구현
2509241 
퀴즈 답안 제출 시 정답, 오답 여부에 따라 유저 정보 업데이트하는 기능이 구현되었습니다.
@moomint8
refactor(Solved Test): Solved 테스트 코드 수정
0a58ad4 
반복해 사용되는 값의 경우 상수 처리를 하고, DB DDL 변경 등의 조건에 따라 적합하게 테스트 코드를 개선했습니다.
@moomint8
Merge branch 'develop' of https://github.com/Triumers/Newsnippet-Back …
84055f3 
…into user
@Leegiyeon
Merge pull request #29 from Triumers/user
9e1520c 
[Feature] 퀴즈 답안 제출 시 유저 정보 업데이트 기능 구현
@moomint8
refactor(Solved): Solved userId token 이용 처리
1b08c9f 
userId를 프론트로부터 제공받는 것이 아닌, 로그인한 사용자의 토큰을 이용해 값을 반환하도록 개선했습니다.
@moomint8
Merge branch 'develop' of https://github.com/Triumers/Newsnippet-Back …
f0860bb 
…into user
@noctesilente
Merge pull request #30 from Triumers/user
fcbda63 
[Refactor] Solved userId token 이용 처리
@moomint8
feat(Security): Header에 UserRole 추가, 권한에 따른 페이지 접근 설정
d49ac45 
UserRole을 헤더에 담아 전달하여 프론트에서 처리할 수 있도록 하였습니다.
비정상적인 접근을 차단하기 위해 SpringSecurity에서도 처리를 구현했습니다.
@moomint8
Merge branch 'develop' of https://github.com/Triumers/Newsnippet-Back …
348c06b 
…into user
@noctesilente
Merge pull request #31 from Triumers/user
f174c26 
[Feature] 유저 권한 헤더에 추가 및 권한에 관한 SpringSecurity 구현
@moomint8
fix(Security): UserRole CORS 처리 추가
5a99e31 
UserRole이 CORS 처리가 되지 않아 받아지지 않던 문제를 해결했습니다.
@moomint8
Merge branch 'develop' of https://github.com/Triumers/Newsnippet-Back …
c8b202b 
…into user
@noctesilente
Merge pull request #32 from Triumers/user
e0dd704 
[Fix] UserRole CORS 문제 해결
@moomint8
build(Newsnippet): CI/CD Docker, jira
21a1b9c 
작업된 CI/CD 관련 파일들을 병합했습니다.
@moomint8 moomint8 merged commit a854826 into main Apr 17, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Sosohy Sosohy Awaiting requested review from Sosohy

@Leegiyeon Leegiyeon Awaiting requested review from Leegiyeon

@kyulin-Kim kyulin-Kim Awaiting requested review from kyulin-Kim

@noctesilente noctesilente Awaiting requested review from noctesilente

Assignees

@Sosohy Sosohy

@Leegiyeon Leegiyeon

@kyulin-Kim kyulin-Kim

@moomint8 moomint8

@noctesilente noctesilente

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@moomint8 @Sosohy @Leegiyeon @kyulin-Kim @noctesilente