-
Notifications
You must be signed in to change notification settings - Fork 133
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Flow] More specific error message if a flow action fails during tick execution #718
Comments
Not all widgets have an automatically generated name, ie. only widgets that are referenced in the generated source code get an automatically generated name. So, we need some other way to identify such a widget. One idea is to call |
This will indeed give an explicit pointer to the place in generated code and will certainly help with pinpointing the problem. |
Implemented. Now, error message looks like this:
Two additional info are now present:
With component path you can find component in Studio by using Find Project Component from Edit menu: |
The reason why I added "Find Project Component" is because not always additional source location will be present, for example if error is in LVGL action. |
Working for me! Thank you. |
Is your feature request related to a problem? Please describe.
This is not a bug, but with current generic error messages generated in code by studio (in screens.c) it is very inconvenient
to determine the widget / place where the error occurs.
This is how it looked like after recent #706 testing in my project:
This text currently exist 135 times in my code.
It would be very handy if the text would be something like:
"Failed to evaluate Text in Label widget obj105"
Even if the widget is not explicitly named in studio it is still much easier to determine the exact widget having the problem.
Describe the solution you'd like
Add the object name in the error message.
Describe alternatives you've considered
Manual searching/deduction. Not very successful with large projects.
Additional context
Add any other context or screenshots about the feature request here.
The text was updated successfully, but these errors were encountered: