Tim Krones
Tim Krones
@shadinaif Thank you for this contribution! Let us know when the changes are ready for review.
Hey @shadinaif, just checking in to see if you're still planning to continue working on this PR?
@shadinaif Sounds good, thanks for the update.
Hey @shadinaif, do you have any updates about when you'll be able to come back to this PR?
Hi @thomty! Just a quick update that we'll be closing this PR shortly because it is stale. You may reopen this pull request, or open a new one, when you...
Hi @shadinaif! Just a quick update that we'll be closing this PR shortly because it is stale. You may reopen this pull request, or open a new one, when you...
Hey @dyudyunov! While https://github.com/openedx/openedx-demo-course/pull/38 is waiting for review (left another [ping](https://github.com/openedx/openedx-demo-course/pull/38#issuecomment-1373701725) there just now), could you please fix merge conflicts here? CC @mphilbrick211 @mariajgrimaldi
Thanks @dyudyunov! I'm marking this as blocked on other work again -- technical review for https://github.com/openedx/openedx-demo-course/pull/38 is complete now (cf. https://github.com/openedx/openedx-demo-course/pull/38#pullrequestreview-1238989191) but that PR also [needs product review](https://github.com/openedx/openedx-demo-course/pull/38#issuecomment-1378477862). CC @mphilbrick211...
@ihor-romaniuk If the target branch ([openedx:open-release/maple.master](https://github.com/openedx/edx-platform/tree/open-release/maple.master)) has changed since you last pushed your changes, perhaps you could try rebasing and pushing them again? CC @mphilbrick211 -- I came here from...
@mphilbrick211 @ihor-romaniuk I'm not sure what's going on with the first check (`CI / check migrations`) -- there's no command line output to check for specific errors under [Details](https://github.com/openedx/edx-platform/actions/runs/5693782764/job/15433615447?pr=31055). However,...