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

[BG-132]: 워크스페이스 생성 API 기능 구현 (3h / 5h) #4

Merged
merged 7 commits into from
Jun 25, 2024

Conversation

GGHDMS
Copy link
Member

@GGHDMS GGHDMS commented Jun 25, 2024

Why

워크스페이스 생성 기능 구현

How

단순하게 현재는 생성만 구현하였음
그래서 예상보다 시간이 덜 걸렸다

Result

image

Prize

서비스 로직에서 단순히 객체를 생성해주고 있는데
이게 맞을지 생각해 봐야 될듯...

Link

BG-132

@GGHDMS GGHDMS requested a review from Dltmd202 June 25, 2024 11:17
@GGHDMS GGHDMS self-assigned this Jun 25, 2024
Copy link
Member

@Dltmd202 Dltmd202 left a comment

Choose a reason for hiding this comment

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

고생했엉!ㅎㅎ

import org.springframework.web.bind.annotation.RequestBody
import org.springframework.web.bind.annotation.RestController

@RestController("/api/v1")
Copy link
Member

Choose a reason for hiding this comment

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

내 기억에 RequestMapping으로 들어갔던 것 같은뎅!
확인부탁행!

Copy link
Member Author

Choose a reason for hiding this comment

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

👍

Comment on lines 3 to 9
enum class Role(
var key: String,
var value: String,
) {
MEMBER("ROLE_MEMBER", "Member"),
LEADER("ROLE_LEADER", "Leader"),
}
Copy link
Member

Choose a reason for hiding this comment

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

나중에 유저쪽 Role이랑 워크스페이스쪽 Role이랑 같이 등장할 일이 있을텐데 enum 이름이 같으면 충돌이 생길 것 같은데
어때??

Copy link
Member Author

Choose a reason for hiding this comment

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

👍

@GGHDMS GGHDMS requested a review from Dltmd202 June 25, 2024 11:52
Copy link
Member

@Dltmd202 Dltmd202 left a comment

Choose a reason for hiding this comment

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

아주 좋아용~

Comment on lines +3 to +9
enum class WorkspaceRole(
var key: String,
var value: String,
) {
MEMBER("ROLE_MEMBER", "Member"),
LEADER("ROLE_LEADER", "Leader"),
}
Copy link
Member

Choose a reason for hiding this comment

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

💯

@GGHDMS GGHDMS merged commit 4f8a2ca into develop Jun 25, 2024
1 check passed
@GGHDMS GGHDMS deleted the feature/BG-132-create-workspace branch June 25, 2024 11:54
@GGHDMS GGHDMS changed the title [BG-132]: 워크스페이스 생성 API 기능 구현 (실제소요시간3 / 스토리포인트5) [BG-132]: 워크스페이스 생성 API 기능 구현 (3h / 5h) Jul 10, 2024
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