-
Notifications
You must be signed in to change notification settings - Fork 76
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
feature: support of error parser for expense report export #113
Conversation
Coverage Report
|
Coverage Report
|
Coverage Report
|
netsuitesdk/errors/parser.py
Outdated
from .helpers import decode_project_or_customer_name, replace_numbers | ||
|
||
|
||
def expense_report_error_parser(error_dict, ns_client, message): |
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.
once we move ahead with other module and see the pattern
will make this function generic too.
Coverage Report
|
netsuitesdk/api/expense_reports.py
Outdated
res = self.ns_client.upsert(er) | ||
try: | ||
res = self.ns_client.upsert(er) | ||
except Exception as e: |
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.
I don't think we should use broad exception
Coverage Report
|
Coverage Report
|
…to parse-netsuite-error
Coverage Report
|
Coverage Report
|
Coverage Report
|
Coverage Report
|
Coverage Report
|
netsuitesdk/api/expense_reports.py
Outdated
@@ -97,5 +100,6 @@ def post(self, data) -> OrderedDict: | |||
er['entity'] = self.ns_client.RecordRef(**(data['entity'])) | |||
|
|||
logger.debug('able to create er = %s', er) | |||
res = self.ns_client.upsert(er) | |||
res = self.ns_client.upsert(er, 'er') |
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.
lets use full name instead of short forms
res = self.ns_client.upsert(er, 'er') | |
res = self.ns_client.upsert(er, 'expense_report') |
Coverage Report
|
Coverage Report
|
Coverage Report
|
Coverage Report
|
Coverage Report
|
Coverage Report
|
No description provided.