Skip to content
This repository has been archived by the owner on Sep 20, 2024. It is now read-only.

CreatePlugin: Get next version helper #5242

Merged
merged 7 commits into from
Jul 5, 2023

Conversation

iLLiCiTiT
Copy link
Member

Changelog Description

Implemented helper functions to get next available versions for create instances.

Additional info

The helper functions are doing what they can do with available data on instances to find next possible available version for an instance. I would be scared to use the value for publishing as publishing may have different requirements in different cases so it should be always optional. This is a quick PR to unblock custom staging directory PR.

Other possible usages need more brainstorm. About how to use it, before using it everywhere. e.g. to show the version in UI (and possibly change it?) would mean that the value must be in instance data.

Testing notes:

Since the code is not used, it must be faked.

  1. Add a creator that would call self.get_next_versions_for_instances on collect_instances and on create.
  2. Validate that the functions are not crashing and are returning right values.

@ynbot
Copy link
Contributor

ynbot commented Jul 4, 2023

Task linked: OP-6324 get next version in creators

@ynbot ynbot added type: feature Larger, user affecting changes and completely new things size/S Denotes a PR changes 100-499 lines, ignoring general files labels Jul 4, 2023
openpype/pipeline/create/utils.py Outdated Show resolved Hide resolved
openpype/pipeline/create/utils.py Outdated Show resolved Hide resolved
Copy link
Member

@jakubjezek001 jakubjezek001 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works as described. LGTM ;)

@iLLiCiTiT iLLiCiTiT merged commit e64328b into develop Jul 5, 2023
@iLLiCiTiT iLLiCiTiT deleted the feature/OP-6324_get-next-version-in-creators branch July 5, 2023 09:54
@ynbot ynbot added this to the next-patch milestone Jul 5, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
size/S Denotes a PR changes 100-499 lines, ignoring general files type: feature Larger, user affecting changes and completely new things
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants