Issue <--> GitHubIssue 간 변환 함수 구현

[JonghoKim-jj]

현재 GitHub GraphQL API 에서 요청하여 받은 결과는 GitHubIssue 로 변환된 후 sled DB 에 저장되는 것으로 알고 있습니다.
Database::insert_issues() 를 통해 삽입할 때는 GitHubIssue 구조체를 넣는데,
Database::issues() 를 통해 이슈를 조회할 때는 Issue 구조체를 받습니다.

impl From<Issue> for GitHubIssue 및 Issue::from(owner: &str, repo: &str, github_issue: GitHubIssue) 를 구현한다면 테스트 코드 등에서 편하게 쓸 수 있을 것 같습니다.

[JonghoKim-jj]

변환함수는 확실한 사용처가 생길 때 구현하면 좋을 것 같습니다.

• Issue -> GitHubIssue 변환은 불필요해 보입니다. end user 에게 제공하는 타입을 내부 DB 에서 사용하는 타입으로 굳이 변환해야할 이유가 없습니다.
• GitHubIssue -> Issue 변환도 아직은 사용처가 명확치 않습니다. 당장 구현해야할 필요가 없을 듯 합니다.
• GitHubIssue -> Issue 변환에는 owner, repo 에 대한 추가 정보가 필요하여, From trait 의 사용은 부적절해보입니다. 만약 필요하다면 Issue::new(owner: &str, repo: &str, github_issue: GitHubIssue) 를 구현하면 좋을 듯 합니다.

p.s. 현재 Issue::try_from_key_value() 가 구현되어 있는데, unused function 인 것으로 보입니다.

[danbi2990]

GitHubIssue -> Issue 변환에는 owner, repo 에 대한 추가 정보가 필요하여

이 부분은 저도 API가 불편하고 DB에 저장된 key를 parsing 과정이 비효율적이라고 생각하여 예전에 변경한 적이 있습니다. 당시 소피와 논의했을 때 별도의 issue로 작업하자는 결론이 났던 것으로 기억하네요.

p.s. 현재 Issue::try_from_key_value() 가 구현되어 있는데, unused function 인 것으로 보입니다.

Issue::try_from_key_value()는 TryFromKeyValue trait을 구현하는데 사용되고, 이는 load_connection 함수에서 열거형 구조체를 만들 때 DB type에서 API type으로 변환하는 과정을 추상화하는데 사용됩니다.

[danbi2990]

완료되어 닫습니다.