feat: 외부링크 유효성검사/랜딩페이지 조회/삭제 API 구현 #297
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- 유효하지 않은 URL에 대한 e2e 테스트 추가 - 검증로직 추가
- 랜딩페이지 접속시 프로젝트의 링크 리스트를 반환하는지 확인하는 E2E테스트 추가 - 프로퍼티 이름을 link에서 linkList로 변경
- 프로젝트의 링크를 반환하는 레포지토리 구현 - 프로젝트의 링크를 반환하는 서비스 구현
- initLandingDto에서 linkDto클래스 만들어 link의 데이터 형식을 만들도록 구현 - 웹소켓 게이트웨이에서 접속시 link정보 함께 반환하도록 구현
- 링크 삭제 서비스 구현 - 테스트 추가 - 링크 삭제 레포지토리 구현
- 웹소켓 게이트웨이에서 링크 삭제 요청시 삭제 후 브로드캐스팅하도록 구현 - 링크삭제 요청 DTO 추가
kimsj-git
reviewed
Jun 17, 2024
Comment on lines
+11
to
+39
| function IsURL() { | ||
| return function (object: Object, propertyName: string) { | ||
| registerDecorator({ | ||
| name: 'IsURL', | ||
| target: object.constructor, | ||
| propertyName: propertyName, | ||
| options: { message: 'invalid url format' }, | ||
| validator: { | ||
| validate(value: any) { | ||
| const URLPattern = new RegExp( | ||
| '^(https?:\\/\\/)' + | ||
| '((([a-z\\d가-힣]([a-z\\d가-힣-]*[a-z\\d가-힣])*)\\.?)+[a-z가-힣]{2,}|' + | ||
| '((\\d{1,3}\\.){3}\\d{1,3}))' + | ||
| '(\\:\\d+)?' + | ||
| '(\\/[-a-z\\d%_.~+가-힣]*)*' + | ||
| '(\\?[;&a-z\\d%_.~+=-가-힣]*)?' + | ||
| '(\\#[-a-z\\d_가-힣]*)?$', | ||
| 'i', | ||
| ); | ||
| return URLPattern.test(value); | ||
| }, | ||
| }, | ||
| }); | ||
| }; | ||
| } | ||
|
|
||
| class Link { | ||
| @IsString() | ||
| @IsURL() |
kimsj-git
reviewed
Jun 17, 2024
Comment on lines
242
to
268
| const expectCreateLinkAndReturnFirstLinkId = (socket, url, description) => { | ||
| return new Promise<void>((resolve, reject) => { | ||
| socket.on('landing', async (data) => { | ||
| const { content, action, domain } = data; | ||
| expect(domain).toBe('link'); | ||
| expect(action).toBe('create'); | ||
| expect(content?.description).toBe(description); | ||
| expect(content?.id).toBeDefined(); | ||
| expect(content?.url).toBe(url); | ||
| expect(content?.description).toBe(description); | ||
| socket.off('landing'); | ||
| resolve(content.id); | ||
| }); | ||
| }); | ||
| }; | ||
|
|
||
| const expectDeleteLink = (socket, linkId) => { | ||
| return new Promise<void>((resolve, reject) => { | ||
| socket.on('landing', async (data) => { | ||
| const { content, action, domain } = data; | ||
| expect(domain).toBe('link'); | ||
| expect(action).toBe('delete'); | ||
| expect(content?.id).toBe(linkId); | ||
| socket.off('landing'); | ||
| resolve(); | ||
| }); | ||
| }); |
There was a problem hiding this comment.
socket.once()를 사용하면 socket.off('landing') 부분을 제거할 수도 있지 않을까요?
There was a problem hiding this comment.
좋은아이디어라고 생각합니다! 즉시 적용할게요~
kimsj-git
reviewed
Jun 17, 2024
Comment on lines
92
to
93
| if (result) return true; | ||
| else return false; |
There was a problem hiding this comment.
이 부분을 이렇게 바꿔도 간결하고 의미 전달이 잘 될거같은데 어떻게 생각하시나요?🤔
return result ? true : falseThere was a problem hiding this comment.
삼항연산자는 항상 생각이 잘 안나는 것 같아요. 좋다고 생각합니다! 바로 적용할게요~
- socket.on후 socket.off로 이벤트의 리스너를 지우는 방식을 socket.once로 이벤트 발생시 바로 한번만 호출되고 리스너가 없어지도록 변경 - 삼항연산자를 사용해 더 짧고 가독성있게 변경
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🎟️ 태스크
(백엔드) 외부 링크 추가 API에 대한 유효성 검사
(백엔드) 랜딩페이지 접속시 외부링크 정보 전송하기
(백엔드) 외부 링크 삭제 API 구현
✅ 작업 내용
🖊️ 구체적인 작업
외부 링크 추가 API에 대한 유효성 검사
랜딩페이지 접속시 외부링크 정보 전송하기
외부 링크 삭제 API 구현
🤔 고민 및 의논할 거리