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

✨ feat: RSS등록 API 연동 #108

Merged
merged 4 commits into from
Nov 19, 2024
Merged

✨ feat: RSS등록 API 연동 #108

merged 4 commits into from
Nov 19, 2024

Conversation

jungmyunggi
Copy link
Collaborator

🔨 테스크

Issue

RSS등록 시 URL유효성 검사는 어떻게 진행해야할까?

  • 각 블로그 마다 RSS URL양식이 달라서 어떤 식으로 유효성 검사를 진행해야할지 고민했습니다.
  • 서버측에서 검증을 한번 더 하고 관리자가 승인/거절 하는 방식이라 클라이언트에서 크게 검사를 진행할 필요가 없음
  • 클라이언트측에서는 https:// 식의 기본적인 URL양식인지만 검사 하는것으로 구현함

📋 작업 내용

  • RSS 등록 API 연동

@jungmyunggi jungmyunggi added the ✨ Feature 기능 구현 label Nov 18, 2024
@jungmyunggi jungmyunggi requested a review from junyeokk November 18, 2024 09:26
@jungmyunggi jungmyunggi self-assigned this Nov 18, 2024
Copy link

netlify bot commented Nov 18, 2024

Deploy Preview for bcdenamu ready!

Name Link
🔨 Latest commit 8b78e31
🔍 Latest deploy log https://app.netlify.com/sites/bcdenamu/deploys/673b084cb0c6920008e5d9d7
😎 Deploy Preview https://deploy-preview-108--bcdenamu.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@jungmyunggi jungmyunggi changed the title ✨ feat: 관리자 로그인 페이지 구현 ✨ feat: RSS등록 API 연동 Nov 18, 2024
Copy link
Collaborator

@junyeokk junyeokk left a comment

Choose a reason for hiding this comment

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

커밋 수고하셨습니다 ,, P2 하나만 적용해주시면 될 것 같아요...!

나머지 코멘트들은 확인만 해주세요~~ 고생 많으셨습니다!

@jungmyunggi jungmyunggi merged commit 655d2cf into main Nov 19, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feature 기능 구현
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FE] RSS 등록 요청 API 연동
2 participants