Skip to content

Conversation

@henryde
Copy link
Contributor

@henryde henryde commented Jul 30, 2025

No description provided.

@henryde henryde requested a review from malhussan July 30, 2025 12:14
@henryde henryde force-pushed the feature/tenant-v4 branch 4 times, most recently from 45dff3c to 922fc9d Compare July 31, 2025 12:01
Copy link
Member

@malhussan malhussan left a comment

Choose a reason for hiding this comment

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

lgtm

return nil, nil
}

if res.StatusCode != 200 {
Copy link
Member

Choose a reason for hiding this comment

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

d: should probably also use isSuccessHTTPStatus

@henryde henryde force-pushed the feature/tenant-v4 branch 4 times, most recently from 03ca68f to 776c219 Compare August 1, 2025 07:12
@henryde henryde force-pushed the feature/tenant-v4 branch from 776c219 to f220a8f Compare August 1, 2025 11:24
@henryde henryde merged commit 4012632 into main Aug 1, 2025
14 checks passed
@henryde henryde deleted the feature/tenant-v4 branch August 1, 2025 12:00
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.

3 participants