AY-6740_CSV ingest: version column optional

[ynbot]

Please describe the enhancement you have in mind and explain what the current shortcomings are?

Client have been discussing AOV publishing a bit recently, so this might be a potential workflow for now. But their preference would be to develop a new publishing creator for Tray publisher for passes. But it feels like this CSV workflow isn't far off what I was asking for.

Current problem is that version column is required / hardcoded. This might be couple lines change to enhance the workflow for next available version.

How would you imagine the implementation of the enhancemenent?

No response

Describe alternatives you've considered:

No response

Additional context:

()
(might be a private channel)

This issue was automatically created from Clickup ticket AY-6740

[BigRoy]

I suppose this will need to allow to be an optional attribute:

ayon-traypublisher/client/ayon_traypublisher/plugins/create/create_csv_ingest.py

Line 752 in 9453ba1

version: int = product_item.version

Anyway, as soon as instance.data["version"] is not set, like it would get popped out of here:

ayon-traypublisher/client/ayon_traypublisher/plugins/create/create_csv_ingest.py

Line 808 in 9453ba1

"version": version,

Then it will automatically compute the version of an instance during publishing.
However, it does disallow to publish multiple versions into one product of course. (Which may then need to 're-enable' a certain validation which I think we skipped for csv publishes?)

[robin-ynput]

Then it will automatically compute the version of an instance during publishing.
However, it does disallow to publish multiple versions into one product of course. (Which may then need to 're-enable' a certain validation which I think we skipped for csv publishes?)

@BigRoy if that's OK with you, I'd say let's discuss this concern on the PR directly: #32
To answer quickly, the way this is done now allows you to publish multiple versions into one product, but only 1 of them can be implicit version number, otherwise instances will conflict indeed.

[BigRoy]

Actually - only allowing one implicit sounds perfectly fine if that works. In that case 🎩 hat off to you. Sounds perfectly fine.