Are L117 and L139require(...) really needed? If yes and if they are intended to prevent askQuestionWithMinBond() from reverting if the setup was incorrect, then require(timeout < 365 days, "Timeout has to be smaller than 365 days") should probably be checked too, right? See Realitio v3. Anyway, I don't think this is an issue, but it caught my attention.
Are L117 and L139
require(...)really needed? If yes and if they are intended to prevent askQuestionWithMinBond() from reverting if the setup was incorrect, thenrequire(timeout < 365 days, "Timeout has to be smaller than 365 days")should probably be checked too, right? See Realitio v3. Anyway, I don't think this is an issue, but it caught my attention.