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로 작업 위임 #51

Merged
merged 10 commits into from
Jan 21, 2025
Merged

Conversation

summersummerwhy
Copy link
Collaborator

@summersummerwhy summersummerwhy commented Jan 20, 2025

🔖 연관된 이슈

📂 작업 내용

  1. edge 생성, 삭제 change websocket에서 감지 -> backend로 api 요청 보내는 방식으로 수정
  2. edge 생성시 workspace 안 저장되는 에러 수정

why?
typeorm의 findOneBy 메서드는 기본적으로 관계(entity relationship)를 자동으로 가져오지 않습니다. 이 메서드는 지정된 조건에 해당하는 필드만 조회하며, 관계 필드나 다른 테이블의 데이터는 포함되지 않습니다. 이를 해결하려면 findOne 메서드와 함께 relations 옵션을 사용해야 합니다. relations를 설정하면 원하는 관계 데이터를 명시적으로 포함시킬 수 있습니다.

const existingFromNode = await this.nodeRepository.findOne({
  where: { id: fromNode },
  relations: ['workspace'], // workspace 관계를 포함
});

findOneBy:

  • 조건에 따라 데이터를 찾지만, 관계는 포함하지 않습니다.
  • 간단한 쿼리에서 사용됩니다.

findOne:

  • relations 옵션을 사용할 수 있어 관계 데이터를 명시적으로 가져옵니다.

📑 참고 자료 & 스크린샷 (선택)

📢 리뷰 요구사항 (선택)

@summersummerwhy summersummerwhy added 🐛 Bug 무언가 정상적으로 동작하지 않는 경우 🐧🚀😶‍🌫️ BE 백엔드 관련 이슈/PR 추적 라벨 labels Jan 20, 2025
@summersummerwhy summersummerwhy self-assigned this Jan 20, 2025
@summersummerwhy summersummerwhy marked this pull request as draft January 20, 2025 05:31
@github-actions github-actions bot requested a review from ezcolin2 January 20, 2025 05:31
@summersummerwhy summersummerwhy marked this pull request as ready for review January 20, 2025 08:30
@ezcolin2
Copy link
Collaborator

local에서는 test 통과하는데 CI 환경에서 test가 실패하네요?
CI 환경에서 종속성 설치하는 과정에서 문제가 있지 않을까 생각합니다.

@github-actions github-actions bot merged commit 57a34b2 into develop Jan 21, 2025
6 of 7 checks passed
@github-actions github-actions bot deleted the bug-be-#48 branch January 21, 2025 04:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐧🚀😶‍🌫️ BE 백엔드 관련 이슈/PR 추적 라벨 🐛 Bug 무언가 정상적으로 동작하지 않는 경우
Projects
None yet
Development

Successfully merging this pull request may close these issues.

엣지 생성, 삭제 api 처리
2 participants