Add ``targets`` field to an effects entry.

[agaertner]

I think entries in the effects array still need a "targets" field because it can happen that the Move itself targets the other team but at the same time a condition like "Evading" is self-applied. So an effect entry may benefit from a targets field as well: Values for it would be: "All", "Self", "Ally", "Team", "Other Team", default: "Target"

default would be Target. The other values make the condition independent of the moves actual target. Don't know if moves like that already exist but they can/will in the future.

Another thing is that all effects seems to be of type "condition". I do not think there will ever be another type for it so I think that field can be safely removed. Note that it is also ambigious because every effect has a field called "condition" meaning we refer to every effect as condition so I think a "type" field is not required here.