Plots2: 将 comment_render_body 助手移动到 Comment 模型中

创建于 2018-10-01 · 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

最后，可以更新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

最有用的评论

我想解决这个问题，这将是我的第一个开源贡献之一。在打开拉取请求之前，我还有什么需要知道的，比如提交消息的特殊模式？

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通知@github.com
写道：

@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 助手移动到 Comment 模型中

最有用的评论

所有4条评论

相关问题