# 211025 TIL

장기현장실습

PLSnote 이슈

#2785 노트기록 저장 또는 대화방 대화 전송시 알림창 기능 반영 이슈 (완료) 

• 노트 기록 시 (완료)

  

  • ko.yml, en.yml: confirm_create_new_note 추가 
  • 저장 버튼 클릭 시에 연결된 "data-action": "activity#activitySubmit"이 있어서 해당 메서드를 수정하여 알림창이 뜨도록 구현 
    • 노트를 생성할 때 필수적으로 입력해야하는 필드가 있어서 필수 필드가 모두 작성된 후에, 저장과 관련된 알림창이 뜨도록 해야했음. 알림창에서 확인을 누르면 노트가 저장되고, 취소를 누르면 아무 액션이 없도록 구현

      activitySubmit(event) {
      	let confirm_result = confirm(this.data.get("confirmCreateNewNote")); 
      	event.preventDefault(); 
      
      	if (this.dueDateTarget.value.length == 0) { 
      		alert(this.data.get("dateMessage")) 
      		this.dueDateTarget.focus() 
      	} else if (this.titleTarget.value.length == 0) { 
      		alert(this.data.get("titleMessage")) 
      	} else if (this.contentTarget.value.length == 0) { 
      		alert(this.data.get("contentMessage")) 
      	} else if (this.hasConfidenceScoreTarget && (this.confidenceScoreTarget.value <= 0 || this.confidenceScoreTarget.value > 10)) { 
      		alert(this.data.get("confidenceScore")) 
      	} else { 
      		if (confirm_result) { 
      			this.submitButtonTarget.setAttribute('disabled', true);
                  this.hashtagFieldTarget.value = this.hashtagFieldTarget.value.trim().replace(/\s+/g, ", "); 
                  this.formTarget.submit()
                  localStorage.removeItem('editor_'+this.data.get("userId"))
              } else {
              	return;
              	}
      	} 
      
      }

• 해당 이슈를 해결한 후, 수정님께 코드 리뷰를 받음 
  • If-else 문 사용시, 더 자주 사용하는 방식으로 수정하면 좋을 것 같다고 피드백을 주심 
    • 내가 작성한 else return;은 있어도 아무 작동을 하지 않는 코드이기 때문에 삭제 
    • confirm 함수를 변수로 선언하는 과정을 스킵 
• 이후, merge request 보낸 상태로 마무리

#2629 전략 네비게이션 개선 의견 이슈 파악 

• 현재 전략, 세부 전략, 프로젝트 명을 클릭해야 메뉴바 리스트를 볼 수 있었다. 그러면서 이름을 클릭할 때 마다 오른쪽 content 내용이 렌더되었는데, 펼치기 버튼을 추가하여 이 문제를 해결하고자 한다. 버튼을 클릭했을 때는 비동기적으로 메뉴바 리스트만 바뀌게 수정하고, 실제 content 내용을 보고 싶을 때는 기존대로 메뉴(전략, 세부 전략, 프로젝트) 이름을 클릭하면 나타나게 한다. 
• 프로젝트를 전체 보기 하는 기능은 오른쪽에 버튼을 추가해 하나의 전략 단위의 리스트를 볼 수 있게 구현할 예정이다. 

 

프론트엔드 개발환경 학습하기

프론트엔드 개발환경의 이해와 실습 (24/39)

[인프런] 프론트엔드 개발환경의 이해와 실습 (린트 Lint)