PLpeopleLayer added to /people page

[sagarpreet-chadha]

Fixes #5874 .

• PR is descriptively titled 📑 and links the original issue above 🔗
• 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 📁
• screenshots/GIFs are attached 📎 in case of UI updation
• ask @publiclab/reviewers for help, in a comment below

Thanks!

[sagarpreet-chadha]

[sagarpreet-chadha]

@jywarren ...how to wait for js to load in capybara ?
wait_until is depreciated .

[sagarpreet-chadha]

Okay increasing Capybara.default_max_wait_time to 180 solved it :)

The screenshot from test looks like this :

https://camo.githubusercontent.com/bdc398875d822623551856f71483176a07fcd56e/68747470733a2f2f73746f726167652e676f6f676c65617069732e636f6d2f706c6f7473322d73637265656e73686f74732f353936312d746573745f4c424c445f677269645f69735f6c6f616465645f6f6e5f70656f706c655f6d61702e706e67

@jywarren ...should we merge this ?

[jywarren]

Hi, @sagarpreet-chadha -- when you click a grid square, what do you see? In the demo here: https://publiclab.github.io/leaflet-blurred-location-display/examples/index.html

... you just see how many markers. Sorry, just catching up here - in this implementation, do you see a list of people instead of a count of markers?

[sagarpreet-chadha]

Hi @jywarren ,
No we do not show the list of people name , we show count of markers . If we wish to show the former , we can do that in next release of LBLD ?
What do you say ?

[jywarren]

That sounds great. Let's wait here until that release! Thanks, Sagarpreet!

…

On Tue, Jul 9, 2019 at 11:26 AM Sagarpreet Chadha ***@***.***> wrote: Hi @jywarren <https://github.com/jywarren> , No we do not show the list of people name , we show count of markers . If we wish to show the former , we can do that in next release of LBLD ? What do you say ? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#5961?email_source=notifications&email_token=AAAF6J6FERLXECMTG6G2OYLP6SUYNA5CNFSM4H4J27OKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZQT5KA#issuecomment-509689512>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAAF6JZ3GGWNXDKBU3URVJTP6SUYNANCNFSM4H4J27OA> .

[jywarren]

Hi @sagarpreet-chadha would you mind linking to an issue in LEL for this? Thanks!

[stale]

Hi 😄, this issue has been automatically marked as stale because it has not had recent activity. Don't worry you can continue to work on this and ask @publiclab/reviewers to add "work in progress" label 🎉 . Otherwise, it will be closed if no further activity occurs in 5 days -- but you can always re-open it if you like! 💯 Thank you for your contributions 🙌 🎈.

[jywarren]

Hi! just noting this now exists via the LEL people layer! Thank you!!!!