M. Tayyab Tahir Qureshi
M. Tayyab Tahir Qureshi
@farhan please assign this ticket to me.
> Hi @ttqureshi -- I'm working my way through the PR backlog for this repo, apologies for letting this one go stale! Could you resolve the conflicts so I can...
conflicts resolved
Thanks @pomegranited for the heads-up! I’ll run the tests locally with tox to catch any errors before pushing. Appreciate the tips.
> So close @ttqureshi ! Just need a little more code coverage -- the easiest way to spot the missing lines is to look for the inline codecov warnings [like...
@pomegranited I hope it's done now :)
@irtazaakram please assign this ticket to me
@kdmccormick [`test_number_mongo_calls`](https://github.com/ttqureshi/edx-platform/blob/995261a8371616e33315ba2a9cd31edd2223f821/xmodule/modulestore/tests/test_mongo_call_count.py#L151) this test case is failing with the test params expecting 37 mongo calls as in _2nd, 3rd_ and _4th_ test case parameters. With the flag (`USE_EXTRACTED_LTI_BLOCK`) set to...
@farhan Thanks for giving the pointers. > 1. We are not adding the [EmptyDataRawMixin](https://github.com/openedx/edx-platform/blob/15caa9746fe59960409a8daa09f24840f2524952/xmodule/raw_block.py#L86) in the extracted XBlock. > 2. `EmptyDataRawMixin` has `data` String field which is missing in the...
@mphilbrick211 YES, it's a WIP. It is currently blocked on [this](https://github.com/openedx/XBlock/issues/823). [This](https://github.com/openedx/xblocks-contrib/issues/99) is the new story for the #XBlock/823.