feature/196 사용자 닉네임 설정에 제한사항을 건다.

[YejiAhn]

Resolves #196

• 닉네임 길이 관련 제한사항을 백엔드, 프론트엔드 각각에서 걸어주었고
• 숫자/영어/한글 로만 이루어진 닉네임을 설정해야한다는 제약사항을 프론트에 걸어주고
• 관련된 css들 손보았습니다.

리뷰는 월요일까지 부탁드려요 🙌

[hwookim]

밤 늦게까지 고생하셨습니다!!👍
되게 간단하게 생각했는데, 생각보다 구현하려니 복잡하군요.
textfield rules의 결과값에 바로 btn disable을 바인딩할 수 있으면 좋을텐데요...
쉽지 않네요 😢

몇가지 리뷰 남겼으니 확인 부탁드려요 ㅎㅎ🥰

[hwookim]

const NICKNAME_RULES = {
  VIOLATED: {
    rule: (newVal) => !!newVal.match(NICKNAME_REGEX),
    message: "닉네임은 숫자/한글/영어로 이루어져야 합니다.",
  },
  UNCHANGED: {
    rule: (newVal, oldVal) => newVal !== oldVal,
    message: "새로운 닉네임을 입력해주세요.",
  },
};

export default {
  name: "MemberUpdateModal",
  ...,
  computed: {
    rules() {
      return Object.values(NICKNAME_RULES).map(
        (value) =>
          value.rule(this.newNickname, this.member.nickname) || value.message,
      );
    },
  },
  ...
  methods: {
    nicknameUpdateDisabled() {
      const validation = Object.values(NICKNAME_RULES).filter(
        ({ rule }) => !rule(this.newNickname, this.member.nickname),
      );
      return validation.length !== 0;
    },
  },

이런 방식은 좀 별로려나요...?
앞으로 룰의 추가가 좀 더 쉬워지겠지만, 지금 당장 코드가 좀 복잡하네요ㅠㅠ
좀 더 고민해볼게요!

[YebinK]

수고했어요 예지니어스! 👍
꽤 무거운 작업이었군요!

몇 가지 리뷰 남겼으니 확인 부탁해요 🙂

[chws]

수고하셨습니다 예지니어스
생각보다 작업이 세세하게 많이 들어갔네요.

하나 논의하고 싶은 사항이, 닉네임의 제한사항(숫자, 영어, 한글만 사용가능)을 백엔드에서는 현재 검사하지 않고 있더라고요. 백엔드에서도 유효성 검사를 해줘야 하지 않나 싶네요... 추가하면 어떨까요? 특수문자가 포함된 닉네임과 같이 유효하지 않은 닉네임을 걸러내야 할 것 같습니다!

[chws]

수고하셨습니다~!

[hwookim]

어... 뭔가 프로젝트 전체적으로 문제가 많이 생긴 듯 하군요...ㅠㅠㅠㅠ
전체적인 교통 정리가 한번 필요할 것 같네요...😢

간단한 리뷰 몇가지 남겼어요. 확인부탁드려요!

[YebinK]

수고했어요 예지니어스 👍
rules가 도입되다니 너무 기뻐요!

테코톡과 겹쳐 고생이 많았을 것 같네요. 이만 approve합니다~

[YebinK]

엔티티 validation에 관한 테스트는 어떻게 하나 궁금했는데
repository test를 하는 방법이 있군요 👍

[LTTTTTE]

고생하셨습니다~
커밋줄수가 1만줄이넘네요 🐰 ㅋㅋㅋ
pull rebase 한번 해주세요~