Magnus Wolffelt
Actually I think the correct solution would be this? `criteria = Sequelize.and(criteria, extraSearchCriteria);` Or something like that, but I'm not very familiar with the Sequelize boolean thingies.
While I agree modifying the reduce opration could be elegant, why is the `Sequelize.and` approach not desirable? It seems to work for me while testing with modified code here.
Fair enough. I can probably make a PR to fix this once I clean up my app code.
I think I figured out a reason to go for the `and` solution - it is more future proof and less bug prone in the face of maintenance. If any...
I agree this would be nice.
I was about to introduce a random library like https://github.com/davidbau/seedrandom and fixed seeds for each challenge, so that the user spawning is always the same for each challenge. However, that...
Do you think it needs to be handled in code mirror or in the input element?
Though isn't the special case just for not interfering with typing of special characters? I suppose you don't write a lot of special characters in JavaScript code normally, right?
Browsers seem to capture this event on a very high level, probably need keypress event handler outside codemirror.
Also add challenge requireUserCountWithinMovesAndTime due to https://github.com/magwo/elevatorsaga/issues/66