Skip to content

refactor: 블로그 포스팅 기능 리펙토링 #160

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.

Sign up for GitHub

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

Jump to bottom
Merged
kakusiA merged 6 commits into from
Sep 22, 2025
Merged

refactor: 블로그 포스팅 기능 리펙토링 #160

kakusiA merged 6 commits into from
Sep 22, 2025

Conversation

@kakusiA
Copy link
Collaborator

@kakusiA kakusiA commented Sep 22, 2025 •
edited
Loading

📝 작업 내용

  • id,password 하드코딩 제거 -> request파라미터에 주입
  • posting된 게시글 url 응답에 추가

🔗 관련 이슈

  • Closes #이슈번호
  • Related to #이슈번호

💬 추가 요청사항

✅ 체크리스트

코드 품질

  • 커밋 컨벤션 준수 (feat/fix/docs/refactor 등)
  • 불필요한 코드/주석 제거

테스트

  • 로컬 환경에서 동작 확인 완료
  • 기존 기능에 영향 없음 확인

배포 준비

  • 환경변수 추가/변경사항 문서화
  • DB 마이그레이션 필요 여부 확인
  • 배포 시 주의사항 없음

dktkf and others added 5 commits September 20, 2025 13:23
@dktkf
refactor:
3e3f36f 
- Fastapi blog/publish url 리펙토랑
  1. 블로그 아이디 비번 하드코딩 -> request parameters로 주입으로 변경
  2. 티스토리 블로그 하드코딩 -> 블로그 역시 parameters로 주입 으로변경
@dktkf
Merge branch 'develop' into feature/blog-refactoring
212862d
@dktkf
feat: 블로그 포스트 기능 생성후 생성된 posting url 추가 로직 개발
6dbe53b
@kakusiA
Merge branch 'develop' into feature/blog-refactoring
6ad5ad3
@kakusiA
fix: 블로그 response 사용하지않는 코드 삭제
3572747
@kakusiA kakusiA marked this pull request as ready for review September 22, 2025 05:06
@kakusiA kakusiA closed this Sep 22, 2025
@kakusiA kakusiA reopened this Sep 22, 2025
@kakusiA kakusiA self-assigned this Sep 22, 2025
@kakusiA kakusiA merged commit 5b9de14 into develop Sep 22, 2025
6 of 7 checks passed
@kakusiA kakusiA deleted the feature/blog-refactoring branch September 22, 2025 05:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@kakusiA kakusiA

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@kakusiA @dktkf