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

Extenal API getUser edge case handling #704

Merged
merged 1 commit into from Apr 12, 2021
Merged

Conversation

@chrmod
Copy link
Member

@chrmod chrmod commented Apr 12, 2021

In case user is not logged in - the API was not sending a response as the getUser promise was rejecting.

In case user is not logged in - the API was not sending a response as the `getUser` promise was rejecting.
@chrmod chrmod requested a review from christophertino as a code owner Apr 12, 2021
@christophertino christophertino added this to the 8.5.6 milestone Apr 12, 2021
@christophertino christophertino merged commit 25a0044 into develop Apr 12, 2021
1 check passed
1 check passed
@github-actions
lint-and-test
Details
@christophertino christophertino deleted the chrmod-patch-1 branch Apr 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants