-
Notifications
You must be signed in to change notification settings - Fork 12
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
salesforce faculty verification #95
Conversation
8079651
to
e459a76
Compare
security_token=settings.SALES_SECURITY_TOKEN) | ||
|
||
|
||
def faculty_status(accounts_id): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Leaving a reminder to myself to add error handling here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah - there was a question from Kathi about what would happen if SF was down - how will the app handle that? This should be addressed in the error handling.
0742d51
to
a656820
Compare
logger.error("{0}('{1}')".format(error_type,error_message)) | ||
return False | ||
|
||
def faculty_status(self,accounts_id): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add spaces after the commas in this file to maintain readability?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure thing. I'll make a PEP8 update
I am getting the following error when I hit /api/user:
I see you added a template to local.py.example - how do I get the keys to test this? |
I just sent you the keys using lastpass. |
This is just the basic functionality of connecting to salesforce. Several features still need to be implemented, .e.g. updating the authentication method to use OAuth 2.0, tracking salesforce usage statistics,
encrypted keys for travis testing,etc.