gravitystorm left a comment (openstreetmap/openstreetmap-website#6606)

I'm happy to see this as an option.

The control should be on the "edit profile" page, not on the profile itself. I 
want to minimise the differences between what you see when you view your own 
profile, compared to what other people see. We have a page to control what is 
or is not shown on your own profile, so that's where the control should be.

I don't think it's worth storing a UserPreference. The UserPreference system is 
really just there as an API, for editors etc to store user-specific state 
between instances, or perhaps to read website preferences that might be useful 
for other apps to know via the API. But I admit that's not really borne out in 
practice, since we've been putting random website preferences in their 
regardless of whether they need to be exposed via the API.

I think there's no need for anything other than the website to read/write this 
particular option, so I don't think it needs to go into the UserPreference 
system, but I'd ask for feedback from other people on this point.

For admins - why do they need to see it? Should this be visible to moderators 
too? I would think moderators are more focussed on mapping behaviour than 
admins, but I don't know if either group use the heatmap as part of their tasks.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/6606#issuecomment-3637976117
You are receiving this because you are subscribed to this thread.

Message ID: 
<openstreetmap/openstreetmap-website/pull/6606/[email protected]>
_______________________________________________
rails-dev mailing list
[email protected]
https://lists.openstreetmap.org/listinfo/rails-dev

Reply via email to