Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -41,3 +41,5 @@ out/

/sample_data/**
!/sample_data/.gitkeep

.DS_Store
2 changes: 1 addition & 1 deletion docker/Dockerfile
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
# 베이스 이미지
FROM openjdk:17-jdk-alpine
FROM openjdk:17-jdk-slim

# 작업 디렉토리 설정
WORKDIR /app
Expand Down
4 changes: 2 additions & 2 deletions docker/docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ services:
dockerfile: docker/Dockerfile
container_name: mywork-application
ports:
- "8080:8080"
- "8081:8080"
environment:
- SPRING_PROFILES_ACTIVE=local
deploy:
Expand Down Expand Up @@ -66,7 +66,7 @@ services:
image: grafana/grafana:11.5.6
container_name: grafana
ports:
- "3000:3000"
- "3001:3000"
volumes:
- ../grafana-data:/var/lib/grafana
environment:
Expand Down
2 changes: 1 addition & 1 deletion k6/member/MemberLoadTest.js
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ export default function() {

let response = loginMember({
email: user.email,
password: user.password
password: '1234'
});

check(response, {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ public class CompanyImageService {
private final CompanyRepository companyRepository;
private final CompanyImageFileHandler companyImageFileHandler;

@Transactional(readOnly = true)
@Transactional
public CompanyImageUploadUrlIssueResponse issueCompanyImageUploadUrl(final UUID companyId, final String fileName) {
final Optional<Company> companyOptional = companyRepository.findById(companyId);

Expand Down
Original file line number Diff line number Diff line change
@@ -1,19 +1,19 @@
package kr.mywork.domain.member.repository;

import java.util.List;
import java.util.Optional;
import java.util.UUID;

import kr.mywork.domain.company.service.dto.response.MemberDetailResponse;
import kr.mywork.domain.member.model.Member;
import kr.mywork.domain.member.service.dto.response.MemberSelectResponse;

import java.util.List;
import java.util.Optional;
import java.util.UUID;

public interface MemberRepository {
Optional<Member> findByEmailAndDeletedFalse(String email);

List<Member> findMemberByCompanyId(UUID companyId, int page, int memberPageSize);
List<Member> findMemberByCompanyId(UUID companyId, int page, int memberPageSize, String memberName);

long countByCompanyIdAndDeletedFalse(UUID companyId);
long countByCompanyIdAndDeletedFalse(UUID companyId,String memberName);

Member save(Member member);

Expand Down
29 changes: 14 additions & 15 deletions src/main/java/kr/mywork/domain/member/service/MemberService.java
Original file line number Diff line number Diff line change
@@ -1,16 +1,5 @@
package kr.mywork.domain.member.service;

import java.time.format.DateTimeFormatter;
import java.util.List;
import java.util.UUID;
import java.util.stream.Collectors;

import org.springframework.beans.factory.annotation.Value;
import org.springframework.context.ApplicationEventPublisher;
import org.springframework.security.crypto.password.PasswordEncoder;
import org.springframework.stereotype.Service;
import org.springframework.transaction.annotation.Transactional;

import kr.mywork.common.auth.components.dto.LoginMemberDetail;
import kr.mywork.domain.activityLog.listener.eventObject.ActivityLogCreateEvent;
import kr.mywork.domain.activityLog.listener.eventObject.ActivityLogDeleteEvent;
Expand All @@ -25,6 +14,16 @@
import kr.mywork.domain.member.service.dto.response.MemberSelectResponse;
import kr.mywork.interfaces.member.controller.dto.request.ResetPasswordWebRequest;
import lombok.RequiredArgsConstructor;
import org.springframework.beans.factory.annotation.Value;
import org.springframework.context.ApplicationEventPublisher;
import org.springframework.security.crypto.password.PasswordEncoder;
import org.springframework.stereotype.Service;
import org.springframework.transaction.annotation.Transactional;

import java.time.format.DateTimeFormatter;
import java.util.List;
import java.util.UUID;
import java.util.stream.Collectors;

@Service
@RequiredArgsConstructor
Expand All @@ -38,18 +37,18 @@ public class MemberService {
private final ApplicationEventPublisher eventPublisher;

@Transactional
public List<CompanyMemberResponse> findMemberByCompanyId(UUID companyId, int page) {
public List<CompanyMemberResponse> findMemberByCompanyId(UUID companyId, int page, String memberName) {

return memberRepository.findMemberByCompanyId(companyId, page, memberPageSize)
return memberRepository.findMemberByCompanyId(companyId, page, memberPageSize, memberName)
.stream()
.map(CompanyMemberResponse::fromEntity)
.collect(Collectors.toList());

}

@Transactional
public long countMembersByCompanyId(UUID companyId) {
return memberRepository.countByCompanyIdAndDeletedFalse(companyId);
public long countMembersByCompanyId(UUID companyId, String memberName) {
return memberRepository.countByCompanyIdAndDeletedFalse(companyId,memberName);
}

@Transactional
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,9 @@ public record CompanyMemberResponse(
String name,
String phoneNumber,
String position,
String department
String department,
String role,
String email

){
public static CompanyMemberResponse fromEntity(Member member) {
Expand All @@ -18,6 +20,8 @@ public static CompanyMemberResponse fromEntity(Member member) {
member.getName(),
member.getPhoneNumber(),
member.getPosition(),
member.getDepartment());
member.getDepartment(),
member.getRole().name(),
member.getEmail());
}
}
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ public void handlePostApprovalAlarmEvent(final PostApprovalNotificationEvent eve

private void saveNotification(final PostApprovalNotificationEvent event) {
notificationService.save(
event.authorId(), event.authorName(), event.postTitle(), event.memberName(), event.memberId(),
event.authorId(), event.authorName(), event.postTitle(), event.actorName(), event.actorId(),
event.targetType(), event.postId(), event.notificationActionType(), event.modifiedAt(), event.projectId(),
event.projectStepId());
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,8 @@
import kr.mywork.domain.notification.model.NotificationActionType;
import kr.mywork.domain.notification.model.TargetType;

public record PostApprovalNotificationEvent(UUID authorId, String authorName, String postTitle, UUID memberId,
String memberName, TargetType targetType, UUID postId,
public record PostApprovalNotificationEvent(UUID authorId, String authorName, String postTitle, UUID actorId,
String actorName, TargetType targetType, UUID postId,
NotificationActionType notificationActionType, LocalDateTime modifiedAt,
UUID projectId, UUID projectStepId) {
}
Original file line number Diff line number Diff line change
Expand Up @@ -33,8 +33,8 @@ private void saveNotification(final CheckListApprovalNotificationEvent event) {
event.authorId(),
event.authorName(),
event.checkListTitle(),
event.authorName(),
event.memberId(),
event.actorName(),
event.actorId(),
event.targetType(),
event.checkListId(),
event.notificationActionType(),
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@
import kr.mywork.domain.notification.model.TargetType;

public record CheckListApprovalNotificationEvent(UUID authorId, String authorName, String checkListTitle,
String memberName, UUID memberId, TargetType targetType,
String actorName, UUID actorId, TargetType targetType,
UUID checkListId, NotificationActionType notificationActionType,
LocalDateTime modifiedAt, UUID projectId, UUID projectStepId) {

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@

import com.querydsl.core.BooleanBuilder;
import com.querydsl.core.types.Projections;
import com.querydsl.core.types.dsl.BooleanExpression;
import com.querydsl.jpa.impl.JPAQueryFactory;
import kr.mywork.domain.company.service.dto.response.MemberDetailResponse;
import kr.mywork.domain.member.model.Member;
Expand Down Expand Up @@ -31,23 +32,41 @@ public Optional<Member> findByEmailAndDeletedFalse(String email) {
}

@Override
public List<Member> findMemberByCompanyId(UUID companyId, int page, int memberPageSize) {
public List<Member> findMemberByCompanyId(UUID companyId, int page, int memberPageSize,String memberName) {
final int offset = (page - 1) * memberPageSize;

return queryFactory
.selectFrom(member)
.where(
member.companyId.eq(companyId),
member.deleted.eq(false))
.orderBy(member.name.asc())
.offset(offset)
.limit(memberPageSize)
.fetch();
.selectFrom(member)
.where(
member.companyId.eq(companyId),
member.deleted.eq(false),
memberNameCondition(memberName)
)
.orderBy(member.name.asc())
.offset(offset)
.limit(memberPageSize)
.fetch();
}

private BooleanExpression memberNameCondition(String memberName) {
if (memberName == null || memberName.trim().isEmpty()) {
return null;
}
return member.name.containsIgnoreCase(memberName);
}

@Override
public long countByCompanyIdAndDeletedFalse(UUID companyId) {
return memberRepository.countByCompanyIdAndDeletedFalse(companyId);
public long countByCompanyIdAndDeletedFalse(UUID companyId,String memberName) {
Long result = queryFactory
.select(member.count())
.from(member)
.where(
member.companyId.eq(companyId),
member.deleted.eq(false),
memberNameCondition(memberName) // 부분 검색 조건
)
.fetchOne();
return result != null ? result : 0L;
}

@Override
Expand Down
Original file line number Diff line number Diff line change
@@ -1,19 +1,5 @@
package kr.mywork.interfaces.company.controller;

import java.util.List;
import java.util.UUID;

import org.springframework.validation.annotation.Validated;
import org.springframework.web.bind.annotation.DeleteMapping;
import org.springframework.web.bind.annotation.GetMapping;
import org.springframework.web.bind.annotation.PathVariable;
import org.springframework.web.bind.annotation.PostMapping;
import org.springframework.web.bind.annotation.PutMapping;
import org.springframework.web.bind.annotation.RequestBody;
import org.springframework.web.bind.annotation.RequestMapping;
import org.springframework.web.bind.annotation.RequestParam;
import org.springframework.web.bind.annotation.RestController;

import jakarta.validation.constraints.Min;
import jakarta.validation.constraints.Pattern;
import kr.mywork.common.api.support.response.ApiResponse;
Expand All @@ -29,18 +15,15 @@
import kr.mywork.domain.member.service.dto.response.CompanyMemberResponse;
import kr.mywork.interfaces.company.controller.dto.request.CompanyCreateWebRequest;
import kr.mywork.interfaces.company.controller.dto.request.CompanyUpdateWebRequest;
import kr.mywork.interfaces.company.controller.dto.response.CompanyCreateWebResponse;
import kr.mywork.interfaces.company.controller.dto.response.CompanyDeleteWebResponse;
import kr.mywork.interfaces.company.controller.dto.response.CompanyDetailWebResponse;
import kr.mywork.interfaces.company.controller.dto.response.CompanyIdCreateWebResponse;
import kr.mywork.interfaces.company.controller.dto.response.CompanyListWebResponse;
import kr.mywork.interfaces.company.controller.dto.response.CompanyNameWebResponse;
import kr.mywork.interfaces.company.controller.dto.response.CompanyNamesWebResponse;
import kr.mywork.interfaces.company.controller.dto.response.CompanySelectWebResponse;
import kr.mywork.interfaces.company.controller.dto.response.CompanyUpdateWebResponse;
import kr.mywork.interfaces.company.controller.dto.response.*;
import kr.mywork.interfaces.member.controller.dto.response.CompanyMemberListWebResponse;
import kr.mywork.interfaces.member.controller.dto.response.CompanyMemberWebResponse;
import lombok.RequiredArgsConstructor;
import org.springframework.validation.annotation.Validated;
import org.springframework.web.bind.annotation.*;

import java.util.List;
import java.util.UUID;

@RestController
@RequiredArgsConstructor
Expand Down Expand Up @@ -141,11 +124,14 @@ public ApiResponse<CompanyNamesWebResponse> companyListOnlyIdNameWebResponseApiR
}

@GetMapping("/{companyId}/members")
public ApiResponse<CompanyMemberWebResponse> getCompanyMember(@PathVariable(name = "companyId") UUID companyId,
@RequestParam(defaultValue = "1") @Min(value = 1, message = "{invalid.page-size}") int page) {
List<CompanyMemberResponse> companyMemberResponses = memberService.findMemberByCompanyId(companyId, page);
public ApiResponse<CompanyMemberWebResponse> getCompanyMember(
@PathVariable(name = "companyId") UUID companyId,
@RequestParam(defaultValue = "1") @Min(value = 1, message = "{invalid.page-size}") int page,
@RequestParam(name = "memberName", required = false) final String memberName
) {
List<CompanyMemberResponse> companyMemberResponses = memberService.findMemberByCompanyId(companyId, page,memberName);

long total = memberService.countMembersByCompanyId(companyId);
long total = memberService.countMembersByCompanyId(companyId,memberName);

List<CompanyMemberListWebResponse> companyMemberWebResponses = companyMemberResponses.stream()
.map(CompanyMemberListWebResponse::fromService)
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,12 +2,12 @@

import kr.mywork.domain.member.service.dto.response.CompanyMemberResponse;

public record CompanyMemberListWebResponse(String id, String name, String phoneNumber, String position, String department) {
public record CompanyMemberListWebResponse(String id, String name, String phoneNumber, String position, String department, String role, String email) {
/**
* 서비스-레벨 DTO(CompanyMemberResponse)를 웹-레벨 DTO로 변환
*/
public static CompanyMemberListWebResponse fromService(CompanyMemberResponse response) {
return new CompanyMemberListWebResponse(response.id().toString(), response.name(), response.phoneNumber(),
response.position(), response.department());
response.position(), response.department(), response.role(), response.email());
}
}
Loading
Loading