Skip to content

Commit e91fb86

Browse files
committed
Use 'add_all' instead of 'add' when committing a list/sequence of tables
1 parent 5190331 commit e91fb86

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/murfey/workflows/notifications/__init__.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ def notification_setup(
2727
existing_notification_parameters = murfey_db.exec(
2828
select(NotificationParameter).where(NotificationParameter.dcg_id == dcgid)
2929
).all()
30-
new_notification_parameters = []
30+
new_notification_parameters: list[NotificationParameter] = []
3131
for k, v in parameters.items():
3232
for enp in existing_notification_parameters:
3333
if enp.name == k:
@@ -44,7 +44,7 @@ def notification_setup(
4444
num_instances_since_triggered=num_instances_between_triggers,
4545
)
4646
)
47-
murfey_db.add(existing_notification_parameters + new_notification_parameters)
47+
murfey_db.add_all(existing_notification_parameters + new_notification_parameters)
4848
murfey_db.commit()
4949
murfey_db.close()
5050
return True

0 commit comments

Comments
 (0)