Skip to content

Conversation

vroldanbet
Copy link
Contributor

No description provided.

@vroldanbet vroldanbet force-pushed the explain-1-30-new-payload-api branch from 7a5aff8 to a61e697 Compare March 18, 2024 10:55
return err
}
debugInfo := &v1.DebugInformation{}
// DebugInformation comes in trailer < 1.30, and in response payload >= 1.30
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

support previous SpiceDB explain API contract and newer.

@vroldanbet vroldanbet force-pushed the explain-1-30-new-payload-api branch from a61e697 to 10a4b08 Compare March 18, 2024 11:01
@vroldanbet vroldanbet force-pushed the explain-1-30-new-payload-api branch from 10a4b08 to 860c91b Compare March 18, 2024 11:31
Copy link
Member

@josephschorr josephschorr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@vroldanbet vroldanbet added this pull request to the merge queue Mar 18, 2024
Merged via the queue into main with commit 8f7c8f0 Mar 18, 2024
@vroldanbet vroldanbet deleted the explain-1-30-new-payload-api branch March 18, 2024 12:19
@github-actions github-actions bot locked and limited conversation to collaborators Mar 18, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants