List of contributors fetched and stored to localStorage

[Rishabh570]

Fixes #53

[Rishabh570]

@jywarren Here is the working GIF

[jywarren]

How does this one relate to #54 ?

Thanks!!

[Rishabh570]

@jywarren #54 is for https://code.publiclab.org#r=all and this is for https://code.publiclab.org/ :)

[Rishabh570]

Both solve one purpose i.e., fetch contributors and store them to localStorage to prevent network calls, but this one is for contributors that show up on https://code.publiclab.org page 😄

[jywarren]

Super. I've merged this. I am a little worried that the code is starting to become difficult to read for newcomers -- do you think some additional comments, or naming functions with names like function doThingWithEach() instead of anonymous function() could help? Or potentially even breaking it up into separate descriptively named files?

Thanks!!! Also watch out that I'm renaming the master branch to main in a moment. All future PRs should be against main. Thank you!!!

[Rishabh570]

@jywarren I completely agree, code readability is also important...I will do whatever is necessary to make the code more readable. I'll create a PR for it...

Also watch out that I'm renaming the master branch to main in a moment

I hope everyone knows this...

[jywarren]

It's ok - any new clones will go from main and any PRs that go to master we can redirect. Thanks!!!

…

On Wed, Oct 10, 2018, 7:53 PM Rishabh Rawat ***@***.***> wrote: @jywarren <https://github.com/jywarren> I completely agree, code readability is also important...I will do whatever is necessary to make the code more readable. I'll create a PR for it... Also watch out that I'm renaming the master branch to main in a moment I hope everyone knows this... — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#61 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AABfJ6Y4e29kg3agf3Ml9L1uWGN1GT2Pks5ujd-KgaJpZM4XL8So> .

[Rishabh570]

Ah, that's great!

[Rishabh570]

@jywarren I've done some refactoring work, the corresponding PR is #63
Please take a look, let me know if anything is needed :)