I like that you're trying
ezpot008! I've taken a break from module work for a while to work on other things, but I still remember some of it. I'll give you some points on where the confusion tends to come from.
Mostly everything in AA2 to reference in triggers is an INT, so you typically want your flows to be set to check for INTs, except for the rare few times you want a FLOAT and the supremely rare STRING. When you int check against a character itself, you're really checking the character's seat number, and the teacher's seat number is always the same (I think it's 24 but check a Teacher module to make sure).
Sometimes you'd even use an int to check for something like a mood, talk option, or group action, and there's the handy enumerate function to let you select all of those conveniently in a list. Some of the confusion comes from the old experienced module devs being so knowledgeable that they skip using enumerate entirely and write the number straight into logic check, which is confusing but effective since copy/pasting trigger conditions always messes up enumerate-placed values.
I've not checked that module for the idea and lack the time now and for a long time ahead sadly, but I wish you luck and I'm happy that you're interested enough to try. There's fun stuff lurking in module triggers. Even if you don't publish any modules you'd know enough that you'll be unpacking triggers, modifying them, and making bizarre cards that have strange effects in gameplay.