search

ynput / ayon-core

Apache License 2.0
22 stars 30 forks source link

Chore: Change log level for missing settings on publish plugin #587

Closed iLLiCiTiT closed 1 month ago

iLLiCiTiT commented 1 month ago

Changelog Description

Log debug for missing settings of plugin instead of warning.

Additional info

After movement we will have a lot of plugins that don't have plugin settings but have filled settings category.

Testing notes:

  1. When Publisher is opened e.g. in maya it should not log a lot of warnings (but lot of debugs)
BigRoy commented 1 month ago

I wonder if it makes sense to still log a warning if the settings category itself does not exist? But only debug log if the plug-in key does not exist (or even ignore that case even more silently?)

iLLiCiTiT commented 1 month ago

wonder if it makes sense to still log a warning if the settings category itself does not exist? But only debug log if the plug-in key does not exist (or even ignore that case even more silently?)

You're probably right, settings category should still warn. Not sure about missing plugin tho...