getavalon / docker

Avalon in a Dockerfile
MIT License
5 stars 2 forks source link

Whitespace Syncing #26

Open tokejepsen opened 6 years ago

tokejepsen commented 6 years ago

Issue

Error when using whitespaces on project names in Kitsu.

Reproducible

  1. Start Avalon and syncing.
  2. Create "Dark Knight" project in Kitsu.
mottosso commented 6 years ago

Assets have the same problem.. I've resorted to deleting whitespaces during the synchronisation, but preferably there would be two names in CGWire; one with restricted characters (for e.g. file-system) and one label.

We should have a chat with Frank about it.

tokejepsen commented 6 years ago

Yeah, modifying the data like deleting the whitespace might cause issue when doing Avalon > CGWire.

tokejepsen commented 6 years ago

Started a discussion for this; https://github.com/cgwire/zou/issues/90

tokejepsen commented 6 years ago

This has been temporarily been fixed by using the assets fix, of replacing whitespace with underscore.

tokejepsen commented 6 years ago

In order to facilitate Avalon > CGWire syncing, we need to fix this issue.

What we could do is assume that the label is the correct name of a project/asset. This way we can more reliably map Avalon to CGWire. Once a solution has been found on the CGWire side, we transition to it.