Skip to content

MBS-14152: Added Livenation Thailand URL Recognition#3665

Open
owlpharoah wants to merge 1 commit intometabrainz:masterfrom
owlpharoah:mbs-14152
Open

MBS-14152: Added Livenation Thailand URL Recognition#3665
owlpharoah wants to merge 1 commit intometabrainz:masterfrom
owlpharoah:mbs-14152

Conversation

@owlpharoah
Copy link
Copy Markdown
Contributor

Problem

Live Nation URLs from the Thai domain (livenationtero.co.th) weren’t being cleaned or validated properly. The cleanup logic didn’t recognize this domain

Solution

Added support for livenationtero.co.th by normalizing it to livenation.co.th in cleanup which automatically redirects to the expected page.

Testing

Tested it locally by adding external event links and works as expected!

@owlpharoah owlpharoah marked this pull request as draft November 2, 2025 05:02
@owlpharoah owlpharoah force-pushed the mbs-14152 branch 3 times, most recently from 1c9d7c9 to a55c8b7 Compare November 2, 2025 17:25
@owlpharoah owlpharoah marked this pull request as ready for review November 2, 2025 18:11
@reosarevok
Copy link
Copy Markdown
Member

Hmm. I see what you are going for here by normalizing it to the non-tero version, but that seems to be a less preferred URL (given the redirect) so it could stop working at some point. Wouldn't it be safer to just keep the livenationtero.co.th link and accept that in the validation? Especially given the validation is already almost entirely separate for this, it might be cleaner that way, you could even separate it entirely based on whether the base url is livenationtero.co.th or livenation.$whatever.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants