Skip to content

Use exceptions_to_problem_details for Actions as well. - #396

Merged
bprobert97 merged 1 commit into
mainfrom
tidy-with-problemdetails
Aug 6, 2026
Merged

Use exceptions_to_problem_details for Actions as well.#396
bprobert97 merged 1 commit into
mainfrom
tidy-with-problemdetails

Conversation

@rwb27

@rwb27 rwb27 commented Aug 6, 2026

Copy link
Copy Markdown
Collaborator

This uses exceptions_to_problem_details from #389 to simplify the function and provide a more complete ProblemDetails response.

It isn't a change of behaviour, just tidies the code slightly.

I looked at doing the same for ActionManager but that isn't straightforward, so I'll keep this PR short and sweet.

This uses `exceptions_to_problem_details` from #389 to simplify the function and provide a more
complete ProblemDetails response.
@barecheck

barecheck Bot commented Aug 6, 2026

Copy link
Copy Markdown

Barecheck - Code coverage report

Total: 97.46%

Your code coverage diff: 0.12% ▴

Uncovered files and lines
FileLines
src/labthings_fastapi/actions.py601-602, 615, 651-652, 847, 988-989, 992, 995, 1043

@bprobert97
bprobert97 merged commit 5054e9f into main Aug 6, 2026
16 checks passed
@bprobert97
bprobert97 deleted the tidy-with-problemdetails branch August 6, 2026 09:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants