Predictive error message when config hasn't been loaded
#3 435 ouverte le 25 mai 2017
Métriques du dépôt
- Stars
- (5 794 stars)
- Métriques de merge PR
- (Aucune PR mergée en 30 j)
Description
I've done support a few times now and one of the most common problems arises from the fact that someone has changed a pack config file but haven't reloaded it.
The error message that usually pops up is a KeyError, since the action or sensor will likely be looking up some key in the config that may not exist in the config stored in the database. However, for some packs, where the default configuration is provided with some dummy values, this may not always be the case
It may be worth putting some thought into how we can detect whether or not the configuration has been reloaded recently, and provide something useful to the user that indicates this is what they should do. At the very least, we should catch "common" exceptions like KeyError and provide this additional context before re-raising.