Matjaz Gregoric

Results 10 comments of Matjaz Gregoric

@ghassanmas Thanks for catching and fixing this issue! @xitij2000 recently finalised his [ruby3 PR](https://github.com/openedx/cs_comments_service/pull/392) -- can you rebase and address my comment above?

Sorry about that @ghassanmas, I forgot to publish the comment, but I published it now. Thanks for rebasing!

I took this, but I don't have permission to assign myself.

The django patch was applied to the `open-release/nutmeg.master` branch on all eligible repos.

@nedbat We forgot to apply the patch to `olive.master` yesterday. Should we schedule another call during which I'll be temporarily granted permissions to apply the patch or is there an...

PRs opened: - https://github.com/openedx/enterprise-catalog/pull/514 - https://github.com/openedx/blockstore/pull/211 - https://github.com/openedx/edx-analytics-data-api/pull/588 - https://github.com/openedx/ecommerce/pull/3850 - https://github.com/openedx/edx-analytics-dashboard/pull/1372 - https://github.com/openedx/xqueue/pull/868 - https://github.com/openedx/ecommerce-worker/pull/188 - https://github.com/edx/enterprise-access/pull/82 - https://github.com/openedx/edx-notes-api/pull/313 - https://github.com/openedx/license-manager/pull/457

:+1: - [x] I read through the code - [x] I checked for accessibility issues - [x] Includes documentation

@xitij2000 This looks great and it required less changes than I thought it would! I think we'll want to pull in the change from https://github.com/openedx/cs_comments_service/pull/395 before merging, but other than...

This is now done, all PRs have been merged into the Olive branch.

Thanks a lot for taking care of this @xitij2000! :+1: - [x] I tested this: I verified that the forums are working correctly on the Olive demo instance. - [x]...