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 #116: Add error handling, when no API Key is found #119

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

kathawala
Copy link

This affects activitites related to OCR and Vision
which have Automagica APIs that can only be accessed
by users who have signed up online and have API keys

This affects activitites related to OCR and Vision
which have Automagica APIs that can only be accessed
by users who have signed up online and have API keys
@tvturnhout
Copy link
Contributor

Thank you @kathawala for your contribution! We believe this is a great addition! Before we are able to merge your pull request into the repository, we require signing of a Contributor License Agreement. This agreement serves to clear out a few legal things. This is only required once, so for any future contributions from you in the future this should no longer be necessary.

Please sign our CLA which can be found here and send it to cla@automagica.com. After we have received the signed document from your side, we can proceed with the pull request.

Comment on lines -9821 to -9823
with open(config_path) as json_file:
local_data = json.load(json_file)
api_key = str(local_data["bot_secret"])

Choose a reason for hiding this comment

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

Its not working for me everytime when i run i am getting Bot secret key error. can you please help it out

@yongruning
Copy link

excuse me,how do I get the API key after registration,will the OCR API limit the number of usage times per day

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

4 participants