Plots2: 重构 profile.html.erb 代码 pt.2

创建于 2018-12-19  ·  4评论  ·  资料来源: publiclab/plots2

第一次?

这是一个仅限初学者的问题。 这意味着我们一直致力于让之前没有为我们的代码库做出贡献的人,甚至之前没有为开源做出贡献的人更容易阅读。

如果这就是您,我们有兴趣帮助您迈出第一步,并且可以像您一样回答问题并帮助您解决问题。 请注意,我们对来自自由和开源软件中代表性不足的团体的贡献特别感兴趣!

我们知道创建拉取请求的过程是新贡献者的最大障碍。 这个问题是给你的💝

如果您之前有过贡献,请考虑将这个留给新人,并查看我们的一般帮助需要的问题。 谢谢!

🤔 你需要知道的。

没有。 本期旨在欢迎您使用开源 :) 我们很高兴引导您完成整个过程。

问题

我们在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

    • [ ] 使用描述问题的唯一名称创建一个新的功能分支

  • [ ] 尝试按照上述步骤解决问题,但即使在完成之前,您也可以:
    将您的更改提交到您的分支并启动拉取请求(请参阅为公共实验室软件做出贡献),但如果您有问题或尚未完成,请将其标记为“进行中”
  • [] 在您的拉取请求正文中引用此问题
  • [ ] 提交拉取请求后,将提供一个额外的清单来合并它

💬 寻求帮助

如果您需要任何帮助 - 这里有一些选择:

first-timers-only help wanted

最有用的评论

我还没有做出开源贡献,所以这会让我感到温暖。 我可以索赔吗?

所有4条评论

我还没有做出开源贡献,所以这会让我感到温暖。 我可以索赔吗?

嗨@dave-light,请继续。 而且,欢迎来到公共实验室和开源世界 :globe_with_meridians: :earth_americas: 。

@dave-light:你在吗? 或者可以声称这一点?

@mzmmlds ,我在做。

此页面是否有帮助?
0 / 5 - 0 等级