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

Zapier - Add better error messaging for 401 responses #6840

Merged
merged 2 commits into from Jun 27, 2023

Conversation

ralewis85
Copy link
Contributor

Description: When a 401 response is given back by Zapier, hint to the end user why that may have occurred

  • If an API Key was initialized with the wrapper, ask them to check their API Key value
  • if an access token was initialized with the wrapper, ask them to check their access token or verify that it doesn't need to be refreshed.

Tag maintainer: @dev2049

@vercel
Copy link

vercel bot commented Jun 27, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
langchain ⬜️ Ignored (Inspect) Jun 27, 2023 7:35pm

Copy link
Contributor

@hwchase17 hwchase17 left a comment

Choose a reason for hiding this comment

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

lgtm, thanks

@hwchase17 hwchase17 merged commit 74848aa into langchain-ai:master Jun 27, 2023
14 checks passed
kacperlukawski pushed a commit to kacperlukawski/langchain that referenced this pull request Jun 29, 2023
)

Description: When a 401 response is given back by Zapier, hint to the
end user why that may have occurred

- If an API Key was initialized with the wrapper, ask them to check
their API Key value
- if an access token was initialized with the wrapper, ask them to check
their access token or verify that it doesn't need to be refreshed.

Tag maintainer: @dev2049
vowelparrot pushed a commit that referenced this pull request Jul 4, 2023
Description: When a 401 response is given back by Zapier, hint to the
end user why that may have occurred

- If an API Key was initialized with the wrapper, ask them to check
their API Key value
- if an access token was initialized with the wrapper, ask them to check
their access token or verify that it doesn't need to be refreshed.

Tag maintainer: @dev2049
aerrober pushed a commit to aerrober/langchain-fork that referenced this pull request Jul 24, 2023
)

Description: When a 401 response is given back by Zapier, hint to the
end user why that may have occurred

- If an API Key was initialized with the wrapper, ask them to check
their API Key value
- if an access token was initialized with the wrapper, ask them to check
their access token or verify that it doesn't need to be refreshed.

Tag maintainer: @dev2049
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.

None yet

2 participants