UmbHost / Our.Umbraco.Forms.uCaptcha

MIT License
5 stars 0 forks source link

Missing directory & filename typo #1

Closed craigs100 closed 2 years ago

craigs100 commented 2 years ago

Installing on a Umb V9.0.1 site with Forms V 9.1.1 installed running on a Linux Mint machine.

Screenshot from 2021-12-06 16-45-04

Attempting to add a uCaptcha type to a Forms Question results in the following set of warnings:-

Screenshot from 2021-12-06 16-46-51

As can be seen from the above:-

  1. The "/Backoffice/Common/SettingTypes/" folder has not been installed.
  2. There is a case typo in the filename "/Backoffice/Common/FieldTypes/uCaptchafield.html" (actually "ucaptchafield.html")
craigs100 commented 2 years ago

PR (#2) raised for item 2. Not actually tested.

craigs100 commented 2 years ago

Have just noticed it's not that SettingTypes isn't uploaded - it's part of UmbracoForms not this package!!!! - it's just a matter of more case typos. This package requests /UmbracoForms/Backoffice/Common/SettingTypes/..... When the actual path is /UmbracoForms/backoffice/Common/SettingTypes/..... Note the lowercase 'b' in backoffice.

Would have done another PR but blocked on the current branch. It's a simple fix though on /Our.Umbraco.Forms.uCaptcha/src/UmbracoForms/uCaptchaField.cs 5 instances on lines 46 to 58