Skip to content
Projects
Groups
Snippets
Help
This project
Loading...
Sign in / Register
Toggle navigation
E
edx-platform
Overview
Overview
Details
Activity
Cycle Analytics
Repository
Repository
Files
Commits
Branches
Tags
Contributors
Graph
Compare
Charts
Issues
0
Issues
0
List
Board
Labels
Milestones
Merge Requests
0
Merge Requests
0
CI / CD
CI / CD
Pipelines
Jobs
Schedules
Charts
Wiki
Wiki
Snippets
Snippets
Members
Members
Collapse sidebar
Close sidebar
Activity
Graph
Charts
Create a new issue
Jobs
Commits
Issue Boards
Open sidebar
edx
edx-platform
Commits
b5718289
Commit
b5718289
authored
Nov 05, 2015
by
Clinton Blackburn
Browse files
Options
Browse Files
Download
Plain Diff
Merge pull request #10525 from edx/clintonb/zendesk-logging
Updated Zendesk error handling
parents
4c934614
bb0effb7
Hide whitespace changes
Inline
Side-by-side
Showing
1 changed file
with
4 additions
and
4 deletions
+4
-4
common/djangoapps/util/views.py
+4
-4
No files found.
common/djangoapps/util/views.py
View file @
b5718289
...
...
@@ -175,8 +175,8 @@ def _record_feedback_in_zendesk(realname, email, subject, details, tags, additio
}
try
:
ticket_id
=
zendesk_api
.
create_ticket
(
new_ticket
)
except
zendesk
.
ZendeskError
as
err
:
log
.
e
rror
(
"Error creating Zendesk ticket:
%
s"
,
str
(
err
)
)
except
zendesk
.
ZendeskError
:
log
.
e
xception
(
"Error creating Zendesk ticket"
)
return
False
# Additional information is provided as a private update so the information
...
...
@@ -184,8 +184,8 @@ def _record_feedback_in_zendesk(realname, email, subject, details, tags, additio
ticket_update
=
{
"ticket"
:
{
"comment"
:
{
"public"
:
False
,
"body"
:
additional_info_string
}}}
try
:
zendesk_api
.
update_ticket
(
ticket_id
,
ticket_update
)
except
zendesk
.
ZendeskError
as
err
:
log
.
e
rror
(
"Error updating Zendesk ticket:
%
s"
,
str
(
err
)
)
except
zendesk
.
ZendeskError
:
log
.
e
xception
(
"Error updating Zendesk ticket"
)
# The update is not strictly necessary, so do not indicate failure to the user
pass
...
...
Write
Preview
Markdown
is supported
0%
Try again
or
attach a new file
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment