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

공지사항 작성 api를 개발했습니다 #300

Merged
merged 7 commits into from
Mar 28, 2024

Conversation

ani2689
Copy link
Member

@ani2689 ani2689 commented Mar 27, 2024

💡 개요

공지사항 작성 api를 개발했습니다.

📃 작업내용

title과 content를 받아 공지사항을 개설합니다.
선생님과 동아리 부장이 아니면 예외가 발생합니다.

🔀 변경사항

🙋‍♂️ 질문사항

🍴 사용방법

🎸 기타

pr을 머지한 후 앱 푸시알람 기능을 추가하겠습니다.

@ani2689 ani2689 added 1️⃣ Priority: High 우선순위 - 상 ✨ Feature 신규 기능 labels Mar 27, 2024
@ani2689 ani2689 self-assigned this Mar 27, 2024
@ani2689 ani2689 linked an issue Mar 27, 2024 that may be closed by this pull request
?: throw ClubNotFoundException()

when(user.roles) {
listOf(Role.ROLE_ADMIN) -> {}
Copy link
Contributor

Choose a reason for hiding this comment

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

이 줄은 없애도 되지 않을까요?

Copy link
Member Author

Choose a reason for hiding this comment

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

d58b9e0 짧게 리팩토링 했어요!

@ani2689 ani2689 merged commit 1c49b9a into develop Mar 28, 2024
1 check passed
@ani2689 ani2689 deleted the feature/299-create-notice branch March 28, 2024 05:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1️⃣ Priority: High 우선순위 - 상 ✨ Feature 신규 기능
Projects
None yet
Development

Successfully merging this pull request may close these issues.

공지 작성 api를 작성하겠습니다.
2 participants