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.

Sign up for GitHub

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

Jump to bottom

Surface 400 error on connection not found when doing a sync status check #1244

Closed
khaliqgant opened this issue Nov 7, 2023 · 0 comments · Fixed by #1245
Closed

Surface 400 error on connection not found when doing a sync status check #1244

khaliqgant opened this issue Nov 7, 2023 · 0 comments · Fixed by #1245
Assignees
@khaliqgant
Labels
bug Something isn't working

Comments

@khaliqgant
Copy link
Member

No description provided.

@khaliqgant khaliqgant self-assigned this Nov 7, 2023
@khaliqgant khaliqgant added the bug Something isn't working label Nov 7, 2023
khaliqgant added a commit that referenced this issue Nov 7, 2023
@khaliqgant
[gh-#1244] use ServiceResponse to surface 400 error
c26e135
@khaliqgant khaliqgant mentioned this issue Nov 7, 2023
Merged
khaliqgant added a commit that referenced this issue Nov 7, 2023
@khaliqgant
[gh-#1244] don't make await for non async and add tests
2b21d66
khaliqgant added a commit that referenced this issue Nov 7, 2023
@khaliqgant
[gh-#1244] remove unrelated changes
d5e5dc7
khaliqgant added a commit that referenced this issue Nov 7, 2023
@khaliqgant
[gh-#1244] remove unrelated
2001522
khaliqgant added a commit that referenced this issue Nov 7, 2023
@khaliqgant
[gh-#1244] use ServiceResponse to surface 400 error (#1245)
442ea4e 
* [gh-#1244] use ServiceResponse to surface 400 error

* [gh-#1244] don't make await for non async and add tests

* [gh-#1244] remove unrelated changes

* [gh-#1244] remove unrelated
khaliqgant added a commit that referenced this issue Nov 7, 2023
@khaliqgant
Gh #1019 dont complain catch (#1246)
eaedeb1 
* [gh-#1244] use ServiceResponse to surface 400 error

* [gh-#1244] don't make await for non async and add tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@khaliqgant khaliqgant

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[gh-#1244] use ServiceResponse to surface 400 error NangoHQ/nango
1 participant
@khaliqgant