fix: Fix rows counter in the Edit Grade modal window (olive branch)

[Lunyachek]

TL;DR - The problem was in the rows counter in the Edit Grades modal window. First digit - number of lines excluding the last line with the form. Second digit - grades data. And our proposal is to include last row with form to common counting

What changed?

FYI: @openedx/content-aurora

[openedx-webhooks]

Thanks for the pull request, @Lunyachek! 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:

• supporting documentation
• Open edX discussion forum threads
• timeline information ("this must be merged by XX date", and why that is)
• partner information ("this is a course on edx.org")
• any other information that can help Product understand the context for the PR

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.

[mphilbrick211]

Hi @Lunyachek! Is this duplicate of #310?

[mphilbrick211]

Hi @Lunyachek! Is this duplicate of #310?

@Lunyachek Nevermind - I see the different titles. :)

[codecov]

Codecov Report

❗ No coverage uploaded for pull request base (open-release/olive.master@85709d9). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files

@@                      Coverage Diff                      @@
##             open-release/olive.master      #311   +/-   ##
=============================================================
  Coverage                             ?   100.00%           
=============================================================
  Files                                ?       109           
  Lines                                ?      1272           
  Branches                             ?       252           
=============================================================
  Hits                                 ?      1272           
  Misses                               ?         0           
  Partials                             ?         0           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

[mphilbrick211]

Hi @mattcarter - just following up on this for review/merge. Thanks!

[mphilbrick211]

Hi @mattcarter - just following up on this for review/merge. Thanks!

My apologies @mattcarter - it looks like Product needs to review first.
CC: @jmakowski1123 @ProductRyan

[jmakowski1123]

Since this fix has multiple PRs associated with it (#310 and #311), I've created a Product Feature ticket. It's currently undergoing Product Review in that Product Feature Ticket.

[mphilbrick211]

Hi @mattcarter @openedx/content-aurora! This is ready for your review. If all looks good, please merge as well. Thank you!

[mphilbrick211]

Hi @mattcarter @openedx/content-aurora! This is ready for your review. If all looks good, please merge as well. Thank you!

Hi @mattcarter - friendly ping on this :)

[mphilbrick211]

HIi @mattcarter and @openedx/content-aurora! Would it be possible for someone to review/merge this? Thanks!

[mphilbrick211]

HIi @mattcarter and @openedx/content-aurora! Would it be possible for someone to review/merge this? Thanks!

Just checking in on this @openedx/content-aurora :)

[openedx-webhooks]

@Lunyachek 🎉 Your pull request was merged! Please take a moment to answer a two question survey so we can improve your experience in the future.