Skip to content
This repository has been archived by the owner on Apr 23, 2024. It is now read-only.

Better error messages #133

Closed
isabellaalstrom opened this issue Sep 14, 2020 · 4 comments · Fixed by #153
Closed

Better error messages #133

isabellaalstrom opened this issue Sep 14, 2020 · 4 comments · Fixed by #153
Labels
enhancement New feature or request

Comments

@isabellaalstrom
Copy link
Contributor

In ha with the component I only get these error messages, no matter what the reason for the 400 is.

requests.exceptions.HTTPError: 400 Client Error: Bad Request for url: http://192.168.1.78:9192/api/objects/tasks

Grocy is however also sending an error_message, like this:

"{\"error_message\":\"SQLSTATE[23000]: Integrity constraint violation: 19 UNIQUE constraint failed: tasks.name\"}"

It would be awesome to get these in the error message from the lib too.
@stale
Copy link

stale bot commented Nov 13, 2020

Is this still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

@stale stale bot added the wontfix This will not be worked on label Nov 13, 2020
@stale stale bot closed this as completed Nov 21, 2020
@isabellaalstrom
Copy link
Contributor Author

I would like to open this again :)

@SebRut SebRut reopened this Feb 10, 2021
@stale stale bot removed the wontfix This will not be worked on label Feb 10, 2021
@SebRut SebRut added the enhancement New feature or request label Feb 10, 2021
@SebRut
Copy link
Owner

SebRut commented Feb 24, 2021

@isabellaalstrom this should be enough to get the error_message from the response body, anything else that would help with errors?

@isabellaalstrom
Copy link
Contributor Author

I'm sure it's fine, I can try to get it to send an error to be sure

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants