Plots2: profile.html.erb 코드 pt.2 리팩토링

처음으로?

이것은 처음 사용 하는 사람 이전에 우리의 코드베이스에 기여한 적이 없는 사람들이나 오픈 소스에 기여한 적이 없는 사람들 이 더 쉽게 읽을 수 있도록 노력했음을 의미합니다.

귀하가 그러한 경우, 저희는 귀하가 첫 번째 단계를 밟는 데 관심이 있으며 질문에 답하고 귀하가 하는 대로 도와드릴 수 있습니다. 우리는 특히 자유 및 오픈 소스 소프트웨어에서 과소 대표되는 그룹의 사람들의 기여에 관심이 있습니다!

우리는 풀 리퀘스트를 생성하는 과정이 신규 기여자에게 가장 큰 장벽이라는 것을 알고 있습니다. 이 문제는 당신을 위한 것입니다 💝

이전에 기여한 적이 있다면 새로운 사람을 위해 이 항목을 남겨두고 일반적인 도움이 필요한 문제를 살펴보십시오. 감사 해요!

🤔 알아야 할 사항.

아무것도. 이번 호는 오픈 소스에 오신 것을 환영합니다 :) 프로세스를 안내해 드리겠습니다.

문제

app/views/users/profile.html.erb 의 코드는 매우 장황합니다... 약간의 리팩토링을 사용할 수 있습니다.

코드에서 다시 발생하는 라인을 리팩토링하는 것으로 시작할 수 있습니다. 현재로서는 첫 번째 줄:

https://github.com/publiclab/plots2/blob/1931439c0c799b3e3420a9dcaace7a54cf895bd4/app/views/users/profile.html.erb#L216 -L223

해결책

무엇으로 변경할 것인가:

"문제" 섹션에 있는 코드 블록의 줄을 리팩터링하여 대신 새 도우미 메서드를 사용합니다.

해당 줄은 <% if current_profile_user %> 로 업데이트할 수 있습니다.

감사 해요!!

수정 단계

• [ ] 아래의 여기에서 댓글로 이 문제를 주장하고 필요한 명확한 질문을 하십시오.
• [ ] 리포지토리를 분기하고 기본 리포지토리 README 지침 https://github.com/publiclab/plots2에 따라 로컬로 설정합니다.
  
  
  
  • [ ] 문제를 설명하는 고유한 이름으로 새 기능 분기를 만듭니다.
  
  
• [ ] 위의 단계에 따라 문제를 해결해 보세요. 하지만 완료되기 전에도 다음을 수행할 수 있습니다.
  변경 사항을 브랜치에 커밋하고 풀 리퀘스트( Public Lab 소프트웨어에 기여 참조)를 시작하지만 질문이 있거나 완료하지 않은 경우 "진행 중"으로 표시하십시오.
• [ ] pull 요청 본문에서 이 문제를 참조하세요.
• [ ] 풀 리퀘스트를 제출하면 병합을 위한 추가 체크리스트가 제공됩니다.

💬 도움 받기

도움이 필요하면 다음과 같은 옵션이 있습니다.

• 아래 댓글
• https://gitter.im/publiclab/publiclab 에서 gitter 채팅에 참여하세요.