Plots2: comment_render_body 도우미를 주석 모델로 이동

에 만든 2018년 10월 01일 · 4코멘트 · 출처: publiclab/plots2

이는되기위한 좋은 후보로 표시되었습니다 첫 번째 타이머 전용 문제 등 이 그것의 간단하고 독립적 인, 그리고 몇 가지 추가 서식, 새로운 기여를위한 좋은 시작점이 될 수 있음을 의미한다. 이 코드에 충분히 익숙하다면 처음 사용하는 사람에게만 해당되는 문제로 형식을 다시 지정하거나 다시 게시 한 다음 @publiclab/reviewers에 핑하여 레이블을 지정하세요. 또는 이번이 처음이 아니라면 스스로 해결해 보세요!

https://github.com/publiclab/plots2/blob/6595ef39f25ca4f2d377cacfd92e86a7460214d6/app/helpers/application_helper.rb#L100 -L107

다음과 같이 https://github.com/publiclab/plots2/blob/master/app/models/comment.rb 에서 주석 방식으로 만들어 보겠습니다.

def render_body

end

그리고 application_helper.rb 에서 제거하십시오!

이 줄도 변경해야 합니다.

https://github.com/publiclab/plots2/blob/6595ef39f25ca4f2d377cacfd92e86a7460214d6/app/views/notes/_comment.html.erb#L120

comment.render_body

마지막으로 https://github.com/publiclab/plots2/blob/master/test/unit/comment_test.rb 의 sanitize comment body 테스트를 업데이트하여 코드를 재현하는 대신 comment.render_body 를 테스트할 수 있습니다. 도우미에서.

이 문제에 도움이 되어 기쁩니다! 처음 사용하는 사람의 문제일 수 있습니다!

Ruby fto-candidate help wanted refactorization

출처

jywarren

가장 유용한 댓글

저는 이 문제를 다루고 싶습니다. 이것은 저의 첫 오픈 소스 기여 중 하나가 될 것입니다. 또한 커밋 메시지에 대한 특수 스키마와 같이 pull 요청을 열기 전에 알아야 할 사항이 있습니까?

mauricetmeyer 에 2018년 10월 01일

🎉1 👍1

모든 4 댓글

mauricetmeyer 에 2018년 10월 01일

🎉1 👍1

아니요, 끌어오기 요청 양식에 템플릿이 있지만 나중에 항상 편집할 수 있으므로 진행하는 동안 도와드릴 수 있습니다. 어서 오십시오! 👍 🎉

jywarren 에 2018년 10월 01일

👍1

@jywarren 감사합니다! 또한 큰 코드베이스를 읽을 시간이 많지 않은 경우가 많기 때문에 이러한 첫 번째 타이머 문제는 코드베이스를 이해하는 데 좋습니다. 👍

mauricetmeyer 에 2018년 10월 02일

당신이 그들을 좋아해서 정말 기쁩니다!!!

2018년 10월 1일 월요일 오후 6:09 Maurice T. Meyer [email protected]
썼다:

@jywarren https://github.com/jywarren 감사합니다! 또한 그 첫 번째 타이머
나는 종종 많은 것을 가지고 있지 않기 때문에 문제는 코드베이스를 알아가는 데 좋습니다.
큰 코드베이스를 읽을 시간. 👍
—
당신이 언급되었기 때문에 이것을 받는 것입니다.
이 이메일에 직접 답장하고 GitHub에서 확인
https://github.com/publiclab/plots2/issues/3552#issuecomment-426080866 ,
또는 스레드 음소거
https://github.com/notifications/unsubscribe-auth/AABfJ7XhfzVY7CxWNvf58mMmOiKBNZjlks5ugpKSgaJpZM4XCx6-
.

jywarren 에 2018년 10월 02일

이 페이지가 도움이 되었나요?

0 / 5 - 0 등급

Plots2: comment_render_body 도우미를 주석 모델로 이동

가장 유용한 댓글

모든 4 댓글

관련 문제