refactor line 216 in profile.html.erb

[dtlight]

Fixes #4342

• [✔️ ] tests pass -- look for a green checkbox ✔️ a few minutes after opening your PR -- or run tests locally with rake test
• [✔️ ] code is in uniquely-named feature branch and has no merge conflicts 📁
• [✔️ ] PR is descriptively titled 📑
• [✔️ ] ask @publiclab/reviewers for help, in a comment below

[dtlight]

Hi @publiclab/reviewer, my build is failing. Travis doesn't show why?

[plotsbot]

	1 Message
📖	@dave-light Thank you for your pull request! I’m here to help with some tips and recommendations. Please take a look at the list provided and help us review and accept your contribution! And don’t be discouraged if you see errors – we’re here to help.

Generated by 🚫 Danger

[oorjitchowdhary]

Checks passed! 🎉
@publiclab/reviewers please review this one

[SidharthBansal]

Current_profile_user can you explain where this function is implemented?

[SidharthBansal]

So that I can verify that it is working properly. Well done both of you. Thanks all

…

On Mon, Dec 24, 2018, 10:05 PM Oorjit Chowdhary ***@***.*** wrote: Checks passed! 🎉 @publiclab/reviewers <https://github.com/orgs/publiclab/teams/reviewers> please review this one — You are receiving this because you are on a team that was mentioned. Reply to this email directly, view it on GitHub <#4391 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AUACQ8dQsiiPld6fbb8r3ySgKt4Xpx0Cks5u8QJLgaJpZM4ZgfnL> .

[dtlight]

Hi @SidharthBansal, I'm a little confused, sorry. Looks like @capedcrusader23 has implemented it in Application controller? Would I need to do this also?

[SidharthBansal]

Just give me the link to the implementation Thanks

…

On Mon, Dec 24, 2018, 10:43 PM Dave Light ***@***.*** wrote: Hi @SidharthBansal <https://github.com/SidharthBansal>, I'm a little confused, sorry. Looks like @capedcrusader23 <https://github.com/capedcrusader23> has implemented it in Application controller? Would I need to do this also? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#4391 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AUACQ52kSRkhY0E-0MMshrLeN7bTucoWks5u8QsugaJpZM4ZgfnL> .

[SidharthBansal]

No need to implement. On Mon, Dec 24, 2018, 10:47 PM Sidharth Bansal < bansal.sidharth2996@gmail.com wrote:

…

Just give me the link to the implementation Thanks On Mon, Dec 24, 2018, 10:43 PM Dave Light ***@***.*** wrote: > Hi @SidharthBansal <https://github.com/SidharthBansal>, I'm a little > confused, sorry. Looks like @capedcrusader23 > <https://github.com/capedcrusader23> has implemented it in Application > controller? Would I need to do this also? > > — > You are receiving this because you were mentioned. > Reply to this email directly, view it on GitHub > <#4391 (comment)>, > or mute the thread > <https://github.com/notifications/unsubscribe-auth/AUACQ52kSRkhY0E-0MMshrLeN7bTucoWks5u8QsugaJpZM4ZgfnL> > . >

[dtlight]

https://github.com/publiclab/plots2/pull/4355/files/6b330f43b5cf279897d5b3d40e3174eeeec1e2af

[SidharthBansal]

This looks good to me.
You can try out help wanted issues from the issue tracker.
Thanks

[grvsachdeva]

Nice work @dave-light! We would love to have your help on other issues, if you are interested in solving more issues, you can find them at https://code.publiclab.org/#r=all