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

Fix unclosed client session #1032

Merged
merged 4 commits into from
Aug 1, 2024
Merged

Conversation

abhahn
Copy link
Member

@abhahn abhahn commented Jul 29, 2024

Motivation and Context

Original PR is here: #830

The goal of this PR is just to fix merge conflicts to enable the change to be merged.

Description

Contribution Checklist

  • I have built and tested the code locally and in a deployed app
  • For frontend changes, I have pulled the latest code from main, built the frontend, and committed all static files.
  • This is a change for all users of this app. No code or asset is specific to my use case or my organization.
  • I didn't break any existing functionality 😄

@abhahn abhahn merged commit 4003f18 into main Aug 1, 2024
9 checks passed
@abhahn abhahn deleted the abhahn/fix-unclosed-client-session branch August 1, 2024 00:21
nikhilnagaraj pushed a commit to Admin-bh-Edge/Edge-Comp-Policies that referenced this pull request Oct 2, 2024
Signed-off-by: Benjamin <[email protected]>
Co-authored-by: Benjamin <[email protected]>
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