Legacy discussion requests not properly serializing course_id

[blarghmatey]

When using the legacy discussions service (cs_comments_service) the in-line discussion block is throwing an error when trying to save the sort_ordering for an authenticated user. This is due to the course_id object getting passed through to the requests parameters unmodified, rather than getting converted to a string.

Relevant Sentry error: https://mit-office-of-digital-learning.sentry.io/share/issue/618f8150704f4a13b0cea9732b35dc56/

This was introduced in the commit 0d4281a and happens when hitting the code block here

[Ali-Salman29]

@blarghmatey

Thank You, we will look into this issue and get back to you soon.

[pdpinch]

Any news about this issue? It's blocking our regular deployment off the main branch and I think it may block Teak.

[taimoor-ahmed-1]

@pdpinch I have opened this PR with the fix according to the error logs reported in this issue. I don't have the legacy cs_comments_service setup at the moment. Will it be possible for you to test with the fix branch? If yes, I can get it merged after your testing.

[blarghmatey]

We deployed your PR to our QA environment and verified that it fixed the issue that we were encountering. Thanks!

[taimoor-ahmed-1]

@blarghmatey I have added the fix in a related PR with other bug fixes too related to discussions. It has been merged into master so you guys can revert to master branch.

Please don't hesitate to reopen this issue if you face a problem with the discussions platform.