fix: rewrite calcualtor tips to reflect the supported features#825
fix: rewrite calcualtor tips to reflect the supported features#825jmyatt merged 1 commit intoopenedx:masterfrom
Conversation
Fixes openedx#820 - Removing constants that are no longer supported - Removing suffixes that are no longer supported
|
Thanks for the pull request, @ghassanmas! I've created OSPR-6457 to keep track of it in JIRA, where we prioritize reviews. Please note that it may take us up to several weeks or months to complete a review and merge your PR. Feel free to add as much of the following information to the ticket as you can:
All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here. Please let us know once your PR is ready for our review and all tests are green. |
|
@ghassanmas Thank you for your contribution. Please let me know once this is ready for our review. |
|
It’s ready for review.
… On 16 Feb 2022, at 16:59, Natalia Berdnikov ***@***.***> wrote:
@ghassanmas <https://github.com/ghassanmas> Thank you for your contribution. Please let me know once this is ready for our review.
—
Reply to this email directly, view it on GitHub <#825 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AD42PH3LLYVYFLTAHNSFH7LU3O3WDANCNFSM5OOVHWFA>.
You are receiving this because you were mentioned.
|
Codecov Report
@@ Coverage Diff @@
## master #825 +/- ##
=======================================
Coverage 83.75% 83.75%
=======================================
Files 253 253
Lines 4415 4415
Branches 1114 1114
=======================================
Hits 3698 3698
Misses 697 697
Partials 20 20
Continue to review full report at Codecov.
|
|
@ghassanmas 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future. |
Fixes #820
@pdpinch Following our conversation, here is the PR.