[BD-14] Connor + Farhaan Sync Problem Editor 9/20

Agenda:

  • Project check in: 

    • What is currently prioritized by opencraft?

      • We have about 150-hours for this (~10/20)

      • Two people working on it

    • What is OC planning to work on?

      • Have a functional new problem xblock editor

      • We want to get this in the main repo

      • The settings widget and OLX Parser are ready for takeoff soon!

    • Where can I see that? Should We specify that in a specific place?

    • Apologies for the slow review time, I have been providing a review for another BD-14 blended project that has a deadline. Need to get that across the line. I have taken a look at the PR, it is just massive and I need to break it up a little bit.

      • @Connor Haugh will work to review in atomic commits.

  • Problem Editor codebase – does it make sense, how we are writing react components? 

  • Questions From @Farhaan Bukhsh :

    • Are we just adding question in the problem xblock editor? because there are label and description and stuff how are we editing that

      • Answer: I think I understand your question. In the existing markdown editor, we have buttons that make “heading” and “description” easy to edit. I’ll pass this question along to John Fay, who has done the UX/UI work for the project, but I would Imagine that we are giving course authors the freedom to write the question however they choose.

        • @Connor Haugh will bring this to JFAY

        • convert question into description + prompt areas that are separate is possible solution

    • Parsing markdown looks vague now because OLX seems to have a lot of rich data we can use, so ditching markdown in favour of OLX looks better

      • Yeah I would tend to agree, the reason we used markdown-> react state was because it was already partially written.

      • This is causing issues for farhaan. Taking olx tags-> react state is very helpful

    • Sending just the markdown to the edx-platform doesn't update the preview so we have to anyhow do a ReactSate to OLX parser as well.

      • yes, this was the second part of the two tickets I shared with Kaustav.

        • @Farhaan Bukhsh will work towards this