Skip to content
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

Add ProblemDetails for invalid policy conditions #773

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

nscuro
Copy link
Member

@nscuro nscuro commented Jul 6, 2024

Description

Adds ProblemDetails for invalid policy conditions.

The frontend can properly interpret ProblemDetails and present a more helpful error message to users than the current generic "request failed" one.

Addressed Issue

N/A

Additional Details

Frontend PR: DependencyTrack/hyades-frontend#87

Checklist

  • I have read and understand the contributing guidelines
  • This PR fixes a defect, and I have provided tests to verify that the fix is effective
  • This PR implements an enhancement, and I have provided tests to verify that it works as intended
  • This PR introduces changes to the database model, and I have updated the migration changelog accordingly
  • This PR introduces new or alters existing behavior, and I have updated the documentation accordingly

@nscuro nscuro added the enhancement New feature or request label Jul 6, 2024
@nscuro nscuro added this to the 5.5.0 milestone Jul 6, 2024
@nscuro nscuro modified the milestones: 5.5.0, 5.6.0 Jul 15, 2024
The frontend can properly interpret `ProblemDetails` and present a more helpful error message to users than the current generic "request failed" one.

Signed-off-by: nscuro <[email protected]>
@nscuro nscuro force-pushed the policy-condition-problemdetails branch from 6246966 to 6f7aa11 Compare August 1, 2024 13:18
@nscuro nscuro modified the milestones: 5.6.0, 5.7.0 Aug 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant