@hlfan commented on this pull request.
>
- if (!o || !d) return;
+ if (!points[0] || !points[1]) return;
Which option would you prefer?
```js
if (!points[0] || !points[1]) return;
if (!(points[0] && points[1])) return;
if (!points.every(p => p)) return;
if (points.some(p => !p)) return;
```
--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/5581#pullrequestreview-2585882597
You are receiving this because you are subscribed to this thread.
Message ID:
<openstreetmap/openstreetmap-website/pull/5581/review/2585882...@github.com>
_______________________________________________
rails-dev mailing list
rails-dev@openstreetmap.org
https://lists.openstreetmap.org/listinfo/rails-dev
