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

Blender: Fix missing animation data when updating blend assets#6165

Merged
simonebarbieri merged 1 commit intodevelopfrom
bugfix/blender-fix_blend_update_missing_animation_data
Jan 25, 2024
Merged

Blender: Fix missing animation data when updating blend assets#6165
simonebarbieri merged 1 commit intodevelopfrom
bugfix/blender-fix_blend_update_missing_animation_data

Conversation

@simonebarbieri
Copy link
Copy Markdown
Contributor

Changelog Description

Fix missing animation data when updating blend assets.

Additional info

There was a specific case when updating assets that were published without animation. If an animation was added after them being loaded, when updating it would throw an error.

Testing notes:

  1. Load an asset that is without any animation data (it must not have any animation in the outline when loaded).
    image
    image
  2. Add an animation to that asset.
  3. Try to update the asset.

@simonebarbieri simonebarbieri self-assigned this Jan 25, 2024
@ynbot ynbot added the size/XS Denotes a PR changes 0-99 lines, ignoring general files label Jan 25, 2024
Copy link
Copy Markdown
Member

@LiborBatek LiborBatek left a comment

Choose a reason for hiding this comment

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

Been tested in Ayon mode and works without any issues! There arent any issues when updating, switching and setting versions of loaded assets whatsoever.

image

@simonebarbieri simonebarbieri merged commit a37f294 into develop Jan 25, 2024
@simonebarbieri simonebarbieri deleted the bugfix/blender-fix_blend_update_missing_animation_data branch January 25, 2024 14:14
@ynbot ynbot added this to the next-patch milestone Jan 25, 2024
@ynbot
Copy link
Copy Markdown
Contributor

ynbot commented Jan 26, 2024

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

host: Blender size/XS Denotes a PR changes 0-99 lines, ignoring general files target: AYON type: bug Something isn't working

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants