Plots2: Переместите помощник comment_render_body в модель комментария

Созданный на 1 окт. 2018 · 4Комментарии · Источник: publiclab/plots2

Это было отмечено как хороший кандидат для став первым старожилы-только вопрос , как это , имея в виду , что это просто, самодостаточным, и с каким - то дополнительным форматированием, может быть большой отправной точкой для нового вкладчика. Если вы достаточно знакомы с этим кодом, рассмотрите возможность его переформатирования или повторной публикации как проблемы , возникающей

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

Наконец, тест sanitize comment body в https://github.com/publiclab/plots2/blob/master/test/unit/comment_test.rb можно обновить для тестирования comment.render_body вместо воспроизведения кода. от помощника.

Рад помочь с этим! Может быть проблема для новичков!

Ruby fto-candidate help wanted refactorization

Источник

jywarren

Самый полезный комментарий

Я хотел бы заняться этим вопросом, это будет один из моих первых проектов с открытым исходным кодом. Также есть ли что-нибудь, что мне нужно знать перед открытием запроса на перенос, например, специальная схема для сообщений фиксации?

mauricetmeyer 1 окт. 2018

🎉1 👍1

Все 4 Комментарий

mauricetmeyer 1 окт. 2018

🎉1 👍1

Нет, в форме запроса на вытягивание есть шаблон, но он всегда доступен для редактирования позже, так что мы можем помочь вам в любое время! Добро пожаловать! 👍 🎉

jywarren 1 окт. 2018

👍1

@jywarren Спасибо! Также эти проблемы с первым таймером очень полезны для знакомства с кодовой базой, так как у меня часто не так много времени, чтобы читать большие кодовые базы. 👍

mauricetmeyer 2 окт. 2018

так рада что они тебе нравятся !!!

В понедельник, 1 октября 2018 г., в 18:09 Морис Т. Мейер [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 2 окт. 2018

Была ли эта страница полезной?

0 / 5 - 0 рейтинги

Смежные вопросы

Добавить подсказку для создания контента во встроенные сетки

first-timers[bot] · 3Комментарии

Убрать повторение

first-timers[bot] · 3Комментарии

Оповещения о функциях на узлах

grvsachdeva · 3Комментарии

Исправить адаптивный стиль кнопки панели инструментов "Вставить заголовок"

noi5e · 3Комментарии

Добавьте дополнительные функциональные тесты для каждого метода в Search API

grvsachdeva · 3Комментарии