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

Checking on the folder entity if it is there. #14

Merged
merged 11 commits into from
Jul 15, 2024

Conversation

moonyuet
Copy link
Member

@moonyuet moonyuet commented Jul 5, 2024

Changelog Description

Make sure the code still workings with/without folder_entity["id"]
Resolve #12

Additional Info

Make sure you build the addon with this branch

Testing Notes

  1. Install addon
  2. Launch Unreal
  3. Load Camera

Copy link
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.

I was able to load Camera(FBX) without any errors shown.

All seems working ok.

@moonyuet moonyuet requested review from BigRoy and LiborBatek July 8, 2024 11:42
Copy link
Contributor

@BigRoy BigRoy left a comment

Choose a reason for hiding this comment

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

Didn't test but changes look good to me.

If I recall correctly from other addons @iLLiCiTiT I'm not sure about version bumping this addon now or whether that should be part of releases instead? (also the version numbers mismatching, one being -dev, etc seems fishy.

@iLLiCiTiT
Copy link
Member

Version bump should not happen!

(and why is there different version in package.py and version.py?)

package.py Outdated Show resolved Hide resolved
@iLLiCiTiT iLLiCiTiT merged commit 79b2bf3 into develop Jul 15, 2024
1 check passed
@iLLiCiTiT iLLiCiTiT deleted the bugfix/Camera-Loading-error branch July 15, 2024 08:57
@antirotor antirotor added the type: enhancement Improvement of existing functionality or minor addition label Oct 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement Improvement of existing functionality or minor addition
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Camera Loading error
5 participants