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

fix: 공지사항 단일 조회 API 토큰 없이 접근 가능하도록 변경 #288

Merged
merged 4 commits into from
Sep 4, 2024

Conversation

leeeeeyeon
Copy link
Collaborator

@leeeeeyeon leeeeeyeon commented Sep 4, 2024

🛰️ Issue Number

#275

🪐 작업 내용

  • 공지사항 단일 조회 API는 웹에서 사용해야 하므로 토큰 없이 접근 가능하도록 SecurityConfig를 수정하였습니다.
  • GitHub Actions 슬랙 알림 전송 시 메시지에 PR 제목만 출력되도록 스크립트를 수정하였습니다.

📚 Reference

✅ Check List

  • DB 스키마가 일치하는지 확인했나요?
  • SonarLint를 반영하여 코드를 수정했나요?
  • merge할 브랜치의 위치를 확인했나요?
  • Label을 지정했나요?

@leeeeeyeon leeeeeyeon added the 🔨 fix Modifying previously implemented features label Sep 4, 2024
@leeeeeyeon leeeeeyeon self-assigned this Sep 4, 2024
@leeeeeyeon
Copy link
Collaborator Author

Overall Project 41.16%

There is no coverage information present for the Files changed

@leeeeeyeon
Copy link
Collaborator Author

Overall Project 41.16%

There is no coverage information present for the Files changed

Copy link

sonarcloud bot commented Sep 4, 2024

@leeeeeyeon
Copy link
Collaborator Author

Overall Project 41.16%

There is no coverage information present for the Files changed

@leeeeeyeon leeeeeyeon merged commit 667417b into develop Sep 4, 2024
2 checks passed
@leeeeeyeon leeeeeyeon deleted the feature/get-notice-remove-auth branch September 4, 2024 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔨 fix Modifying previously implemented features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant