#2027 reinstated a check in spin add that the component being added had the same trigger type as the current ones. This was correct at the time because we then supported only a single trigger type in an app. However, time moves on (spoilers), and an app can now cheerfully (well, moderately cheerfully) have more than one trigger type. So we should remove the check.
Thanks @michelleN for spotting that the check was still there! (I thought I had removed it when we did multi-trigger-type, but I guess that was just a beautiful dream.)
#2027 reinstated a check in
spin addthat the component being added had the same trigger type as the current ones. This was correct at the time because we then supported only a single trigger type in an app. However, time moves on (spoilers), and an app can now cheerfully (well, moderately cheerfully) have more than one trigger type. So we should remove the check.Thanks @michelleN for spotting that the check was still there! (I thought I had removed it when we did multi-trigger-type, but I guess that was just a beautiful dream.)