Muhammad Faraz Maqsood

Results 16 comments of Muhammad Faraz Maqsood

@Danyal-Faheem and I have tested the changes, there's still some python tests failing under `test-python` workflow. We have added pylintrc file to ignore some of the pylint warnings. We also...

Hi @feanil, this PR is ready for review. Can you take a look? Thanks.

> Hi @feanil, this PR is ready for review. Can you take a look? Thanks. Hello @feanil, Hope so you are doing well. A soft reminder for you. So that...

Hello @andrey-canon, Can u also have a look at this PR for me. Thanks.

Hello @feanil and @andrey-canon, I hope you guys are doing well, a reminder for you guys to review this PR for this [issue](https://github.com/openedx/ecommerce/issues/4161) to be closed. Thanks.

Hello @andrey-canon, thanks for the review. I have updated the code according to the requested changes. Can u look into it again so that we can finalize it.

Hello @feanil, Can you also have a look at the backport [PR](https://github.com/openedx/ecommerce/pull/4165) to redwood and merge it. Thanks.

[PR](https://github.com/openedx/ecommerce-worker/pull/254) for ecommerce-Worker is already merged and its [Issue](https://github.com/openedx/ecommerce-worker/issues/262) is closed. For ecommerce, it's [PR](https://github.com/openedx/ecommerce/pull/4153) is up for review. Already asked some folks for a review. Once it's reviewed &...

Closing this issue as the upstream Issues are closed. - https://github.com/openedx/ecommerce/issues/4161 - https://github.com/openedx/ecommerce-worker/issues/262

> @Faraz32123 Are the unit tests complete now? Did you test the new forum v2 with the discussions MFE? - As for unit tests, current edx-platform tests that uses v1...