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

[RELEASE/3.0.0] release: v3.0.0 #254

Merged
merged 35 commits into from
May 9, 2024
Merged

[RELEASE/3.0.0] release: v3.0.0 #254

merged 35 commits into from
May 9, 2024

Conversation

Ji-soo708
Copy link
Member

@Ji-soo708 Ji-soo708 commented May 9, 2024

❓ 기능 추가 배경


3.0차 MVP 기능 배포입니다

➕ 추가/변경된 기능


3.0차 MVP 기능 추가 및 리팩터링

🥺 리뷰어에게 하고싶은 말


한 번에 배포 되게 해주세요... 🙏

CChuYong and others added 30 commits May 9, 2024 23:51
* feat: implementing pick..

* feat: implement pick feature

* feat: add picked members api

* feat: implement picked api

* feat: implement GET api

* feat: check member exists

* feat: update api specs

* feat: concat sql queries

* feat: seperate notification
* refactor: Remove unused api parameter got by parsing token and reformat parameter sequence from MemberController. MemberApi

* refactor: Rename and reformat the methods in MemberController

* refactor: Rename and reformat the methods in MemberSerivce

* refactor: Fix the anti-pattern of MemberController by moving transactional code to MemberService

* refactor: Refactor minor code dirty

* refactor: Refactor legacy MemberControllerTest by removing unnessary tests and spliting to MemberServiceTest due to anti-pattern refactoring

* refactor: Resolve code smell issue, distinguishing AuthenticationFailedException by ErrorCode

* feat: Add MemberServiceTests for projections methods
* feature: add type and mission_id column sql

* feature: add type and mission_id column to Post Entity

* test: fix test due to added column

* feat: change type field's type in Post
* hotfix: add Enumerated annotation

* hotfix: change char to varchar
* feat: add type field in constructor

* feat: add mission post create/read mocking api

* test: fix failed test due to added type field

* style: add comment

* refactor: change type's validate logic in create post

* refactor: change type enum value

* refactor: rename Enum Type

* refactor: change if to swith
* feat: Add HomeApi and getFamilyMemberMonthlyRanking spec

* feat: Add getFamilyMemberMonthlyRanking api response mocking at HomeController
#235)

* feature: add getMissionEligibleStatus and getSurvivalUploadStatus mock api

* refactor: rename api name and dto field
* feat: add view based api

* docs: update view based api docs

* feat: add isMeUploaded

* fix: update birthday evaluate logics
* refactor: change exception name

* feature: add createMissionPost logic

* feature: add searchMissionPost

* refactor: add available parameter in createPost and fetchDailyFeeds

* refactor: seperate validate logic in mission post
* feat: Add leftUploadCountUntilMissionUnlock, dailyMissionConten and Add debug parameter to control boolean flag

* refactor: Integrate HomeAPI to MainViewController and Add getNighttimePage API

* fix: Fix wrongly written get mapping url of getNighttimePage

* fix: Add the missing GetMapping annotation and fix wrongly written swagger summary
* feature: add getRemainingSurvivalPostCount Api

* feature: add getRemainingSurvivalPostCount logic to getDaytimePage Api

* test: add getRemainingSurvivalPostCount Repository test

* test: add getRemainingSurvivalPostCount Api test
* feature: add getSurvivalUploadStatus API logic

* feature: add getMissionAvailableStatus API

* test: add getMissionAvailable Repository Test

* test: add getMissionAvailable and getSurvivalUploadStatus Api Test

* style: change response dto field name

* feature: add getMissionAvailableStatus and getSurvivalUploadStatus logic to getDaytimePage

* refactor: correct anti code

* refactor: add member.deletedAt.isNull condition to isCreatedSurvivalPostByMajority
* refactor: refact calculateRemainingSurvivalPostCountUntilMissuonUnlocked logic

* feature: add isActiveMember method and refact code

* test: refact test code due to seperated logic

* test: add calculateRemainingSurvivalPostCountUntilMissionUnlocked test code
* feat: Add mostRecentSurvivalPostDate in FamilyMemberMonthlyRankingResponse according to requirement

* feat: Add isMeMissionUploadedToday field And Rename IsMeUploadedToday as isMeSurvivalUploadedToday from DaytimePageResponse

* fix: Fix broken PostApiTest and Add new test for added api
* feat: Divide getCalendar API to getMonthlyCalendar and getDailyCalendar And Impl getDailyCalendar

* fix: Add CalendarAPI test code for getDailyCalendar and fix the bug found by test code

* refactor: Remove debug system print code

* refactor: Extract the code to map post to calendar response as the method

* refactor: Extract the code to calculate allFamilyMembersUploaded as the method

* fix: Fix the runtime exception during PostType enum mapping
…247)

* fix: Add the fields of PostResponse in DailyCalendarResponse

* fix: Fix broken CalendarApiTest

* fix: Fix broken CalendarApiTest2
* feat: Add getFamilyMembersMonthlySurvivalRanking API in PostController

* feat: Add mvc test code for new API(getFamilyMembersMonthlySurvivalRanking) with nested test class

* feat: Impl getFamilyMemberMonthlyRanking API at MainViewController

* feat: Add mvc test code for new API(getFamilyMemberMonthlyRanking) at MainViewApiTest

* fix: Add stream filter to exclude familyMember who didnt upload post at that month from getFamilyMembersMonthlySurvivalRanking API of PostController and Add test code to validate it from PostApiTest and MainViewApiTest

* fix: Fix failed broken test code

* fix: Fix failed broken test code2

* refactor: Refactor compareTo method of PostRankerDTO for better readability

* refactor: Add blank line at that last of MainViewApiTest

* refactor: Extract duplicated mapping code from getFamilyMemberMonrhlyRanking method of MainViewController

* fix: Fix IndexOutOfBoundsException from getFamilyMemberMonthlyRanking API
Ji-soo708 and others added 5 commits May 9, 2024 23:55
…248)

* refactor: count family members by family ID logic to count based on yesterday's date

* test: fix broken test due to changed logic

* style: change api summary

* refactor: delete unused method

* refactor: refactor isCreatedSurvivalPostByMajority code

* test: delete unused test
* chore: Add mission module path in the targets of cicd modification checking

* feat: Add DailyMissionRegistrationJob in DailyMissionJob with Spring scheduler and table lock

* feat: Impl the methods of MissionService and DailyMissionHistyoryService those api responses was mocked

* feat: Add exceptions handling at DailyMissionRegistrationJob

* feat: Add DailyMissionHistoryRepositoryTest

* feat: Add MissionRepositoryTest for getRandomMissionExcludingIds

* feat: Add DailyMissionJobTest for registerDailyMission

* feat: Add error handling code for duplicated attempting to create daily mission history and catch exception code

* feat: Add tests for newly added error handling and exception catching code

* refactor: Refactor method and parameter name according to naming convention

* refactor: Delete unnecessary Repository annocation from DailyMissionHistoryRepositoryCustomImpl

* fix: Make Nested test class lower case
* feat: Add getLatestPost API to implement new policy of family member ranking

* fix: Add null handling in getLatestPostAPI

* feat: Make mostRecentSurvivalPostDate return most recent surcicial post date in this month else then return null

* refactor: Refactor nested test class in MianviewApiTest to follow naming convention

* feat: Add test code for findLatestPost in PostRepositoryCustomImpl

* refactor: Add PostType paramter at getRecentPost API

* feat: Add createdAt index at post table
* feature: add validateMissionPostCreateAccessMember logic

* feature: handle Mission Post Access Denied Exceptions

* test: add createMissionPost test

* style: change leftUploadCountUntilMissionUnlock field description

* test: change parameter value in createPost Api test

* test: delete createMissionPost test for normal case

* feature: change exception code

* feat: Restore mistakenly deleted method

* feat: Enable validation code for duplicated family disabled due to iOS buf from on-boarding

---------

Co-authored-by: Kwon770 <[email protected]>
@Ji-soo708 Ji-soo708 changed the title Release/v1.3.0 [RELEASE/1.3.0] release: v1.3.0 May 9, 2024
@Ji-soo708 Ji-soo708 added the 🚀 RELEASE 배포 label May 9, 2024
Copy link

sonarcloud bot commented May 9, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

Copy link

github-actions bot commented May 9, 2024

Test Results

 51 files   51 suites   11s ⏱️
141 tests 141 ✅ 0 💤 0 ❌
142 runs  142 ✅ 0 💤 0 ❌

Results for commit a9e3637.

Copy link

github-actions bot commented May 9, 2024

Code Coverage

File Coverage [55.62%]
AuthorizationFailedException.java 100% 🍏
MissionPostAccessDeniedFamilyException.java 100% 🍏
ErrorCode.java 100% 🍏
MissionPostCreateAccessDeniedMemberException.java 100% 🍏
InvalidMemberNameLengthException.java 100% 🍏
DuplicatedDailyMissionHistoryException.java 100% 🍏
CreateMissionRequest.java 100% 🍏
MonthlyCalendarResponse.java 100% 🍏
MissionUploadStatusResponse.java 100% 🍏
MissionAvailableStatusResponse.java 100% 🍏
FamilyMemberRankerResponse.java 100% 🍏
RemainingSurvivalPostCountResponse.java 100% 🍏
FamilyMemberMonthlyRankingResponse.java 100% 🍏
PostRankerResponse.java 100% 🍏
MissionResponse.java 100% 🍏
SurvivalUploadStatusResponse.java 100% 🍏
DailyMissionHistoryResponse.java 100% 🍏
PostResponse.java 100% 🍏
DailyCalendarResponse.java 100% 🍏
DailyMissionHistoryService.java 100% 🍏
MemberQuitReasonService.java 100% 🍏
WidgetController.java 100% 🍏
ReactionRepositoryCustomImpl.java 100% 🍏
DailyMissionHistoryRepositoryCustomImpl.java 100% 🍏
CommentRepositoryCustomImpl.java 96.55% 🍏
CommentService.java 94.89% 🍏
ReactionService.java 93.62% 🍏
MemberBridgeImpl.java 92.96% 🍏
MemberService.java 92.74% 🍏
PostRankerDTO.java 92% 🍏
PostType.java 91.3% 🍏
DailyMissionJob.java 85.48% 🍏
CalendarController.java 81.8% 🍏
PostRepositoryCustomImpl.java 80.23% 🍏
PostController.java 79.38%
MissionService.java 72.09%
PostService.java 71.56%
MeController.java 61.49%
Member.java 57.72%
MemberController.java 49.61%
FamilyNotificationEventListener.java 47.5%
MissionBridgeImpl.java 45.45%
PostBridgeImpl.java 37.5%
DailyMissionHistory.java 30.5%
MemberDeviceService.java 29.85%
Mission.java 29.66%
Post.java 29.04%
MissionController.java 24.32%
MainViewController.java 22.64%
MemberPickService.java 15%
SpringWebExceptionHandler.java 10.63%
AuthController.java 8.16%
DailyNotificationJob.java 7.24%
DuplicateSurvivalPostUploadException.java 0%
AlreadyPickedMemberException.java 0%
DuplicateMissionPostUploadException.java 0%
DailyMissionHistoryNotFoundException.java 0%
PickFailedAlreadyUploadedException.java 0%
MissionNotFoundException.java 0%
UpdateMissionRequest.java 0%
MainPageFeedResponse.java 0%
MainPageTopBarResponse.java 0%
NighttimePageResponse.java 0%
MainPagePickerResponse.java 0%
DaytimePageResponse.java 0%
DailyMissionResponse.java 0%
MemberPick.java 0%
Total Project Coverage 51.98% 🍏

@Ji-soo708 Ji-soo708 requested review from Kwon770 and CChuYong May 9, 2024 15:10
@Ji-soo708 Ji-soo708 changed the title [RELEASE/1.3.0] release: v1.3.0 [RELEASE/1.3.0] release: v3.0.0 May 9, 2024
@Ji-soo708 Ji-soo708 changed the title [RELEASE/1.3.0] release: v3.0.0 [RELEASE/3.3.0] release: v3.0.0 May 9, 2024
@Kwon770 Kwon770 changed the title [RELEASE/3.3.0] release: v3.0.0 [RELEASE/3.0.0] release: v3.0.0 May 9, 2024
Copy link
Collaborator

@Kwon770 Kwon770 left a comment

Choose a reason for hiding this comment

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

승인합니다!

@Kwon770 Kwon770 merged commit 27f7f27 into main May 9, 2024
3 of 4 checks passed
@Kwon770 Kwon770 deleted the release/v1.3.0 branch May 9, 2024 15:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants