Andy Allan
Andy Allan
Given the above comments, I'm still in favour of this change.
@atomoil Sorry for not reviewing this earlier, it's at the top of my todo list. I hope to get time on Wednesday.
I forgot to say that I'm happy overall with this, and I appreciate that since we don't have anything similar at the moment, it's hard to know what conventions (like...
I had a look at this yesterday, but I ran out of time before having anything to share. For the initializers, we have [config/initializers/banners.rb](https://github.com/openstreetmap/openstreetmap-website/blob/master/config/initializers/banners.rb) that loads data from a yaml...
> I also had a brief look at ways of loading data into the i18n system, but again I don't have anything to share yet. I think the answer to...
> hey @gravitystorm and @tomhughes, > > It's been a couple of weeks, so I just wanted to see if you had any thoughts on the above so that I...
@atomoil thanks for your patience. I think it's got to the stage where I should be making changes myself, rather than constantly asking you to do all the work! There's...
Also needs tests
I've covered most of what I want to do with this PR before the next review, but the big remaining issue is around thread safety. I want to make sure...
> Or is there is there a third direction where you plan to drop the whole blacklist thing, because it is clearly decorative and can be overcome by setting up...