fix: Added maxLength and default tip for email subject

[Inferato]

If the user puts more than 128 symbols in the email subject it causes an error.

Added length restriction and field tip.
Related PR: #154

[openedx-webhooks]

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

[e0d]

@Inferato CI is failing, can you have a look. Also, I've invited you to the triage team so tests will run automatically going forward.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (2de1299) 83.01% compared to head (276da02) 83.01%.
Report is 20 commits behind head on master.

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #154   +/-   ##
=======================================
  Coverage   83.01%   83.01%           
=======================================
  Files          46       46           
  Lines         683      683           
  Branches      132      132           
=======================================
  Hits          567      567           
  Misses        116      116           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[arbrandes]

Would it be too much to ask for a screenshot with the error message showing?

[Inferato]

Would it be too much to ask for a screenshot with the error message showing?

Yeah. I'll add it to the description.

[Inferato]

@arbrandes Screenshot was added. FYI

[arbrandes]

Thank you! But I'd also like to see one with the error pop-up introduced in this PR. Can you attach one?

[mphilbrick211]

Thank you! But I'd also like to see one with the error pop-up introduced in this PR. Can you attach one?

@Inferato - flagging this for you!

[mphilbrick211]

Hi @Inferato! Friendly reminder re: Adolfo's request above. Thanks!

[mphilbrick211]

Hi @Inferato! Friendly reminder re: Adolfo's request above. Thanks!

@Inferato - just following up on this!

[dyudyunov]

@arbrandes @mphilbrick211

Sorry for the delays, Taras is no longer a member of RG.
Currently, I'm gathering all the PRs he worked on. I will check their statuses and update them if necessary.

I will leave the status update comments on each PR's comments.

[dyudyunov]

@arbrandes I switched to Taras' branch and checked the behavior.

His fix adds the form help text and limits the subject input to 128 chars

Notes:

• the help text (Maximum 128 characters) is always displayed (since the page loading)
• according to changes I was not able to send emails with subject length > 128 chars
• I reproduced the issue without Taras' changes on Quince (500 error when there are more than 128 chars)

[arbrandes]

Thanks!

[openedx-webhooks]

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