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

[Refactor/#168] JSON Property Case 변경 (SNAKE_CASE -> LOWER_CAMEL_CASE) #169

Merged
merged 2 commits into from
Oct 31, 2023
Merged

[Refactor/#168] JSON Property Case 변경 (SNAKE_CASE -> LOWER_CAMEL_CASE) #169

merged 2 commits into from
Oct 31, 2023

Conversation

skytin1004
Copy link
Member

@skytin1004 skytin1004 commented Oct 31, 2023 •
edited
Loading

기존 SNAKE_CASE로 설계된 JSON Property의 설정과 JSON 변수들을 전부 LOWER_CAMEL_CASE에 맞게 변경하였습니다.

주요 변경사항

  1. com.inhabas.api > resource-server > main > config > WebConfig 의 PropertyNamingStrategies를 LOWER_CAMEL_CASE로 변경
  2. com.inhabas.api > resource-server > test > resources > application.yml 의 PropertyNamingStrategies를 LOWER_CAMEL_CASE로 변경
  3. 두 변경사항과 관련된 모든 테스트 파일들의 JSON 변수의 이름을 SNAKE_CASE > LOWER_CAMEL_CASE로 변경

모든 테스트 통과, 빌드 이상없음.

@skytin1004 skytin1004 temporarily deployed to build_gradle October 31, 2023 11:30 — with GitHub Actions Inactive
@skytin1004 skytin1004 requested a review from whitem4rk October 31, 2023 11:36
@skytin1004 skytin1004 changed the title [Refactor/#168] JSON [Refactor/#168] JSON Property Case 변경 (SNAKE_CASE -> LOWER_CAMEL_CASE) Oct 31, 2023
@whitem4rk whitem4rk merged commit 14640f6 into InhaBas:master Oct 31, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@whitem4rk whitem4rk whitem4rk approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@skytin1004 @whitem4rk