search

arpitgandhi-xoriant / lti-params-api

0 stars 0 forks source link

looking good #5

Closed dcadams closed 2 years ago

dcadams commented 2 years ago

I made a comment in the code of latest commit.

Things to do: pass course_id as a param if you like that idea pep8 and pylint make installable as a Django app unit tests

RiteshChouhan21 commented 2 years ago

Passing course_id as param and making app installable is done in the latest commit. Please have a look.

arpitgandhi-xoriant commented 2 years ago

Updated code standard using pep8 and pylint.

RiteshChouhan21 commented 2 years ago

All issues are addressed.