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

Changes to persist user if internal request and user was explicitly p… #9085

Merged
merged 1 commit into from Oct 18, 2023

Conversation

hanzlamateen
Copy link
Member

@hanzlamateen hanzlamateen commented Oct 18, 2023

…assed.

Summary

🤖 Generated by Copilot at 6180382

Added user object to async local storage in authenticate.ts hook. This allows other modules to access the user information for logging or error handling purposes.

References

closes #insert number here

Explanation

🤖 Generated by Copilot at 6180382

  • Enable the use of the user object in async local storage by checking and entering it in the authenticate hook (link)

🤖 Generated by Copilot at 6180382

context has user
enter async storage now
autumn of errors

QA Steps

List any additional steps required to QA the changes of this PR, as well as any supplemental images or videos.

Checklist

  • If this PR is still a WIP, convert to a draft
  • When this PR is ready, mark it as "Ready for review"
  • ensure all checks pass
  • Changes have been manually QA'd
  • Changes reviewed by at least 2 approved reviewers

@barankyle barankyle added this pull request to the merge queue Oct 18, 2023
Merged via the queue into dev with commit 027fd46 Oct 18, 2023
12 of 14 checks passed
@barankyle barankyle deleted the persist-internal-user branch October 18, 2023 18:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants