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

Change generate-assertions http response to status only #173

Closed
vteague opened this issue Aug 5, 2024 · 1 comment · Fixed by #175
Closed

Change generate-assertions http response to status only #173

vteague opened this issue Aug 5, 2024 · 1 comment · Fixed by #175

Comments

@vteague
Copy link
Member

vteague commented Aug 5, 2024

See meeting notes - assume the dashboard gets info from the database and instead just give it a 4-valued ENUM to tell it whether assertion generation succeeded or failed, and whether it's worth redoing.

@vteague
Copy link
Member Author

vteague commented Aug 6, 2024

Actually this should apply to #raire-service too - might as well return status only from the raire-service endpoint.

Change GenerateAssertionsResponse to have two booleans: one for whether assertion generation succeeded, and one for whether it is worth retrying.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
1 participant