Infocaster / UrlTracker

An Umbraco package that tracks 404 not found results and helps you redirect visitors to the right page
https://infocaster.net/wat-we-doen/umbraco-plugins/301-url-tracker
MIT License
17 stars 12 forks source link

Validation Error, when Culture = "zh-Hant". Traditional Chinese #150

Closed Artem-Savenkov closed 9 months ago

Artem-Savenkov commented 1 year ago

_Using version: 10.2.1

Steps to reproduce the behavior:

  1. create the redirect object with Culture property equal "zh-Hant". "zh-Hant" is the ISO code for Traditional Chinese. var redirect = new Redirect { Culture = "zh-Hant" } ;
  2. try to Add this object to DB via that method redirect = await _redirectService.AddAsync(redirect);
  3. Get the Validation Exception: the field Culture must match the regular expression '^[a-z]{2}(?:-[A-Z]{2})?$'

Expected behavior successfully added to DB the redirect object with Culture = "zh-Hant".

Screenshots image image

D-Inventor commented 1 year ago

Hi @Artem-Savenkov thank you for reaching out! Your issue is very clear, I'll dive into it and come back to you when I have an update

D-Inventor commented 10 months ago

Hi @Artem-Savenkov We just released version 10.3.1-beta.1 with a fix for this issue. Are you interested to give it a try and see if that fixes the issue?

Artem-Savenkov commented 10 months ago

Hi Denis, yes, this is the good news. I will check this.

On Tue, Oct 17, 2023 at 9:26 AM Dennis @.***> wrote:

Hi @Artem-Savenkov https://github.com/Artem-Savenkov We just released version 10.3.1-beta.1 with a fix for this issue. Are you interested to give it a try and see if that fixes the issue?

— Reply to this email directly, view it on GitHub https://github.com/Infocaster/UrlTracker/issues/150#issuecomment-1765822833, or unsubscribe https://github.com/notifications/unsubscribe-auth/AKJ4NYDQJNVHL2IQFDKAA7LX7YXKJAVCNFSM6AAAAAA2UEB6Q6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONRVHAZDEOBTGM . You are receiving this because you were mentioned.Message ID: @.***>

Artem-Savenkov commented 10 months ago

Hi @D-Inventor , I checked this flow. it works great now. Thanks ;)