Add xblock 2.0 support #359
Conversation
|
@irtazaakram, based on my initial understanding, it seems that in the test_export test case, this line is being called in the edx-platform, which is removing the xblock-family attribute, leading to the assertion error. In the test_import test case, the line is invoking the parse_xml method of edx-sga with the id-generator, causing the error. |
|
Thanks for looking into this @Anas12091101. Seems like a circular dependency. We are updating xblock https://github.com/openedx/edx-platform/pull/34398/files#diff-f82cb2116924880209a8846a4a84aee3f70301cb682fed8e05a987103da61c9a here that depends on this change here. What are our options here |
|
@irtazaakram I have tested it locally. LGTM. Can you tell me how long is it expected to merge your edx-platform PR? |
There was a problem hiding this comment.
Tests are failing but this is required as per @irtazaakram because they need a new release
|
@Anas12091101 PR should be merged as early as possible. I'll let you know as soon its ready. |
Related to openedx/edx-platform#34440