WIP: #12, maybe also #13: change VideoTags parsing a bit to fix some minor issues with Brighteon
#14
Closed
jamesapyrich
wants to merge 1 commits from videotags-updates
into main
Loading…
Reference in New Issue
There is no content yet.
Delete Branch 'videotags-updates'
Deleting a branch is permanent. It CANNOT be undone. Continue?
So! Not sure if this is ready to be merged, and I've not tested it... the issues I'm trying to address is that we shouldn't be splitting tags up by words for videos.
Brighteon in particular has problems with this, as it complains about more than 25 tags, and sometimes splitting tags like this results in duplicate tags, which it also doesn't like.
Some sites also complain about non-Latin chars in the tags, which this change does not address... also, haven't spent any time figuring out if the function defined on lines 1160-1164 should be organized in another file, perhaps an internal libraries/functions file, or what!
Anyway, this hopefully gets us closer to resolving some of the issues I opened around tags.
Step 1:
From your project repository, check out a new branch and test the changes.Step 2:
Merge the changes and update on Gitea.