[openstreetmap/openstreetmap-website] Allow sorting and filtering of notes displayed in user profile (#832)

Andy Allan notifications at github.com
Thu Apr 29 09:39:53 UTC 2021


> I **really** want to make **very sure** I won't be wasting a lot of my time and effort if I decide to try to tackle this one...

It's impossible for me to make any guarantees, since I'm only a volunteer here. I can't predict the future (e.g. if I'll be available if/when you make a pull request) and my available time is limited anyway. So my recommendation is to look at our previous performance and estimate based on that. Most PRs get a review reasonably quickly, but I can only review about 1-3 PRs per week (remember I'm a volunteer too so my time is limited) so some PRs wait for a few weeks before being reviewed. And there are definitely some that take much longer to be dealt with, though they are often more about policy changes than simple code changes. But that's only 35 out of the nearly 1500 PRs so far, so you can see most are dealt with in reasonable time.

If you are worried that it might be a lot of effort for just one PR, then I would encourage you to aim to make multiple PRs! That way the learning of the codebase will be more worthwhile. Remember there are hundreds of thousands of mappers using the website every week, so even small improvements can have a big impact. 

But like all volunteering, you put in the effort in the hope it will be useful, not with any guarantees.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/issues/832#issuecomment-829088867
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20210429/d5216b94/attachment.htm>


More information about the rails-dev mailing list