Re: [FIX] consider user === null

[hochan222]

Re: #11

추가 사항

• 아래 사항들을 구현하기 위해 updateBatch 함수를 Refactoring 했습니다. #10
  • [x] Refresh의 경우 coalition 변경이 빈번하지 않다면 배치에서는 호출 제외
  • [x] Refresh의 경우 coalition 이전 값을 승계. 또는 coalition 개별 업데이트
• sonarqube config #4 길드는 지금까지 4시즌이 있었지만 한번 지정되면 변경이 되지않아서 제외시키는 방향으로 코드를 짰습니다. 나중에 길드가 변경이 된다면 일괄적으로 바꿔주면 좋을것같다고 생각합니다. 이 부분에 대해서는 어떻게 생각하시나요?

수정 사항

• sqlite findOne() 값을 찾지 못했을때 null이 반환되어 user에 null이 할당됩니다.
• 이때, 이전 코드는 user.coalition에 접근을 해서 오류가 나서 수정했습니다.

• 추가적으로 이 오류가 console에 나타나지 않아서 try catch를 추가했습니다.

  갑자스런 42api 구조 변경..? (4월 23일 13시)

  • api에서 받아오는 데이터 값의 구조가 많이 변했습니다... 이에따라 전체 로직을 수정해야할것같습니다. 많이 당황스럽습니다..#12
  • 새로 바뀐 구조에서는 프로젝트별 기간이 존재하지 않습니다...ㅠ 어떻게하면 좋을까요..ㅠ
  • 현재 PR은 42API 바뀌기전을 기준으로 작성된 코드입니다.
  • Before

    {
    "id": 69063,
    "email": "holee@student.42seoul.kr",
    "login": "holee",
    "first_name": "Hochan",
    "last_name": "Lee",
    "usual_first_name": null,
    "url": "https://api.intra.42.fr/v2/users/holee",
    "phone": "hidden",
    "displayname": "Hochan Lee",
    "usual_full_name": "Hochan Lee",
    "image_url": "https://cdn.intra.42.fr/users/holee.jpg",
    "staff?": false,
    "correction_point": 22,
    "pool_month": "february",
    "pool_year": "2020",
    "location": null,
    "wallet": 66,
    "anonymize_date": "2022-04-22T00:00:00.000+09:00",
    "groups": [],
    "cursus_users": [
    {
    "grade": null,
    "level": 9.18,
    "skills": [
    {
      "id": 4,
      "name": "Unix",
      "level": 10.95
    },
    {
      "id": 1,
      "name": "Algorithms & AI",
      "level": 8.44
    },
    {
      "id": 3,
      "name": "Rigor",
      "level": 6.35
    },
    {
      "id": 14,
      "name": "Adaptation & creativity",
      "level": 2.9
    }
    ],
    "blackholed_at": null,
    "id": 85197,
    "begin_at": "2020-01-20T00:00:00.000Z",
    "end_at": "2020-02-14T14:00:00.000Z",
    "cursus_id": 9,
    "has_coalition": true,
    "user": {
    "id": 69063,
    "login": "holee",
    "url": "https://api.intra.42.fr/v2/users/holee"
    },
    "cursus": {
    "id": 9,
    "created_at": "2015-11-04T10:58:13.979Z",
    "name": "C Piscine",
    "slug": "c-piscine"
    }
    },
    {
    "grade": "Learner",
    "level": 5.07,
    "skills": [
    {
      "id": 3,
      "name": "Rigor",
      "level": 4.92
    },
    {
      "id": 2,
      "name": "Imperative programming",
      "level": 4.53
    },
    {
      "id": 17,
      "name": "Object-oriented programming",
      "level": 4.05
    },
    {
      "id": 10,
      "name": "Network & system administration",
      "level": 3.06
    },
    {
      "id": 1,
      "name": "Algorithms & AI",
      "level": 2.62
    },
    {
      "id": 4,
      "name": "Unix",
      "level": 2.2800000000000002
    },
    {
      "id": 5,
      "name": "Graphics",
      "level": 2.25
    }
    ],
    "blackholed_at": "2021-06-18T04:42:00.000Z",
    "id": 90261,
    "begin_at": "2020-02-24T04:42:00.000Z",
    "end_at": null,
    "cursus_id": 21,
    "has_coalition": true,
    "user": {
    "id": 69063,
    "login": "holee",
    "url": "https://api.intra.42.fr/v2/users/holee"
    },
    "cursus": {
    "id": 21,
    "created_at": "2019-07-29T08:45:17.896Z",
    "name": "42cursus",
    "slug": "42cursus"
    }
    }
    ],
    "projects_users": [
    {
    "id": 2082176,
    "occurrence": 0,
    "final_mark": null,
    "status": "in_progress",
    "validated?": null,
    "current_team_id": 3539809,
    "project": {
    "id": 1332,
    "name": "webserv",
    "slug": "webse
    ...

  • After

    {
    data: {
    id: '69063',
    type: 'users',
    attributes: {
    email: 'holee@student.42seoul.kr',
    login: 'holee',
    'first-name': 'Hochan',
    'last-name': 'Lee',
    'usual-first-name': null,
    url: 'https://api.intra.42.fr/v2/users/holee',
    phone: 'hidden',
    displayname: 'Hochan Lee',
    'usual-full-name': 'Hochan Lee',
    'image-url': 'https://cdn.intra.42.fr/users/holee.jpg',
    'staff?': false,
    'correction-point': 22,
    'pool-month': 'february',
    'pool-year': '2020',
    location: null,
    wallet: 66,
    'anonymize-date': '2022-04-22T00:00:00.000+09:00'
    },
    relationships: {
    groups: [Object],
    'cursus-users': [Object],
    'projects-users': [Object],
    'languages-users': [Object],
    achievements: [Object],
    titles: [Object],
    'titles-users': [Object],
    partnerships: [Object],
    patroned: [Object],
    patroning: [Object],
    'expertises-users': [Object],
    roles: [Object],
    campus: [Object],
    'campus-users': [Object]
    }
    }
    }

[hochan222]

@kenu 앞으로 git graph로 확인하겠습니다. 제가.. 브랜치를 망칠뻔 했습니댜..ㅠㅠ

[kenu]

PR보고 저도 여러 아이디어를 얻게 되요. 고맙고 잘 하고 있어요! :) Looks Good To Me!

[hochan222]

감사합니다! 어제는 분명히 42API 구조가 저렇게 받아져왔는데 지금 다시해보니 원래대로 돌아왔습니다.. 좋은 점심 시간 보내세요 :)