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

Add/Improve info for unreal docs #257

Merged
merged 21 commits into from
Oct 2, 2024
Merged

Conversation

moonyuet
Copy link
Member

@moonyuet moonyuet commented Sep 24, 2024

Changelog Description

Improve/Add Info for unreal docs

  • Sequence Hierarchies
  • Connect animation and camera to the sequence loaded from layout
  • Camera
  • Animation
  • Layout
  • UAssets(Admin Docs)

Additional info

n/a

@moonyuet moonyuet marked this pull request as ready for review September 27, 2024 10:10
@LiborBatek
Copy link
Member

@MustafaJafar @moonyuet shouldnt be that publish template already defined in the AYON as defaults? or at least we should do that and only if not present due to older deployment put that template setup section as a solution when not setuped? at least by Note paragraph at the very begining of that template setup section?

@MustafaJafar
Copy link
Contributor

MustafaJafar commented Sep 30, 2024

@MustafaJafar @moonyuet shouldnt be that publish template already defined in the AYON as defaults?

Not yet. Here's the PR for reference ynput/ayon-backend#328

I totally agree with you let's convert it to note.

edit: I've added it in 4249a26

image

Copy link
Contributor

@MustafaJafar MustafaJafar left a comment

Choose a reason for hiding this comment

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

This PR looks cool.

Side note:
I've expressed my personal about the structure of unreal docs. which can be tackled in another PR.

#260

@MustafaJafar MustafaJafar merged commit 1db0cac into main Oct 2, 2024
@MustafaJafar MustafaJafar deleted the documentation/add_unreal_docs branch October 2, 2024 07:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants