Skip to content
This repository was archived by the owner on Sep 16, 2020. It is now read-only.

Comments

Fix bug in notification import#755

Open
rogerxaic wants to merge 1 commit intoansible:masterfrom
rogerxaic:patch-1
Open

Fix bug in notification import#755
rogerxaic wants to merge 1 commit intoansible:masterfrom
rogerxaic:patch-1

Conversation

@rogerxaic
Copy link

When removing a string from a list of strings, del list["string"] will cause a "TypeError: list indices must be integers or slices, not str". list.remove("string") must be used.

We can see this when importing notifications:

File "/usr/local/lib/python3.7/site-packages/tower_cli/cli/transfer/send.py", line 1503, in import_notification_relations
  del existing_notifications[notification_name]
TypeError: list indices must be integers or slices, not str

When removing a string from a list of strings, `del list["string"]` will cause a "TypeError: list indices must be integers or slices, not str".

File "/usr/local/lib/python3.7/site-packages/tower_cli/cli/transfer/send.py", line 1503, in import_notification_relations
  del existing_notifications[notification_name]
TypeError: list indices must be integers or slices, not str
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant