Table Reflection and missing triggers

[theelderbeever]

I am attempting to use table reflection for a read only application that I am developing. I keep getting errors pertaining to ValueError('Exception occurred while generating logs table: No matching trigger found') however, this table an other don't have triggers. Is this something that you have encountered? Additionally, piccolo shouldn't really need to know about triggers on the tables, correct? Is there a way to not reflect those parts?

[dantownsend]

I haven't personally come across it yet, but the error is being raised here:

piccolo/piccolo/apps/schema/commands/generate.py

Line 173 in 570b395

raise ValueError("No matching trigger found")

The code is probably trying to work out the ON UPDATE and ON DELETE for a foreign key, and it's failing for some reason.

I think that bit of code needs changing, so it handles the exception gracefully, and instead outputs a warning.

Until a fix is released, you can patch your local version by changing that line of code, so it returns None instead of raising an exception.

[dantownsend]

Issue #504

[theelderbeever]

Awesome! Thanks for the quick reply!