Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Chore: Fix duplicated project name on create project structure #6166

Merged
merged 2 commits into from
Jan 25, 2024

Conversation

iLLiCiTiT
Copy link
Member

Changelog Description

Small fix in project folders. It is not used same variable name to change values which breaks values on any next loop.

Additional info

Avoid reusing clean_items variable during for loop.

Testing notes:

  1. Keep __project_root__ in folder structure settings.
  2. Create a project and make sure it has more than one root.
  3. Run create folders action (e.g. in Project manager in OpenPype or action in ftrack).

@ynbot
Copy link
Contributor

ynbot commented Jan 25, 2024

Task linked: OP-7514 Multiroot folder structure

@ynbot ynbot added size/XS Denotes a PR changes 0-99 lines, ignoring general files type: bug Something isn't working labels Jan 25, 2024
@iLLiCiTiT iLLiCiTiT changed the title fix project name duplication in project folders Chore: Fix duplicated project name on create project structure Jan 25, 2024
@iLLiCiTiT iLLiCiTiT requested a review from 64qam January 25, 2024 11:47
@64qam 64qam self-requested a review January 25, 2024 13:22
@iLLiCiTiT iLLiCiTiT merged commit 0e6b351 into develop Jan 25, 2024
15 checks passed
@iLLiCiTiT iLLiCiTiT deleted the bugfix/OP-7514_Multiroot-folder-structure branch January 25, 2024 14:01
@ynbot ynbot added this to the next-patch milestone Jan 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/XS Denotes a PR changes 0-99 lines, ignoring general files sponsored Client endorsed or requested target: AYON target: OpenPype type: bug Something isn't working
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

3 participants