search

ynput / ayon-kitsu

Official AYON<->Kitsu intetgration (WIP)
Apache License 2.0
8 stars 6 forks source link

Synced Folder names are made to be lower case #74

Open nikshankarnoble opened 1 month ago

nikshankarnoble commented 1 month ago

Is there an existing issue for this?

Current Behavior:

Right now, when a Folder is synced across from Kitsu it's name is forced to be lower case as part of the slug-ification, so for example an asset cameraA in Kitsu will be created in AYON as cameraa. The label will still be correct as that's just copied straight across.

Expected Behavior:

As AYON Folders support upper case characters in their name, I would expect the name to be synced across to match if the entity name in Kitsu is already a valid name in AYON. This isn't a big deal as we should be linking them through their kitsuId metadata anyway, but it can cause some unnecessary confusion when the asset name in UIs and on disk don't match Kitsu

Version

1.0.0

What platform you are running on?

Linux / Centos

Steps To Reproduce:

  1. Ensure that the Kitsu service is setup in AYON.
  2. Create a new entity in Kitsu with capitals in the name, e.g bobLoblawA
  3. The created Folder in AYON will have a name of bobloblawa

Are there any labels you wish to add?

Relevant log output:

No response

Additional context:

No response

nikshankarnoble commented 1 month ago

@martastain @BigRoy it looks like Tasks have this same behaviour - are you happy for this to be changed to match capitalisation in names for both Tasks and Folders? If so I'm happy to raise a PR for this change! :)

nikshankarnoble commented 1 month ago

Also - unrelated to this issue but a couple of inconsistencies I ran into with the support form for raising this issue (which is great by the way!):