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

Overridden properties link to definition in Godot Docs #44187

Closed

Conversation

joshdegraw
Copy link
Contributor

@joshdegraw joshdegraw commented Dec 8, 2020

This is the in-progress implementation of godotengine/godot-docs#4432

Right now the XML generator has been modified to contain the new data, and I am now in the process of editing makerst.py to format it properly.

@joshdegraw joshdegraw changed the title Overridden Properties Link to Definition in Godot Docs Overridden properties link todDefinition in Godot Docs Dec 8, 2020
@joshdegraw joshdegraw changed the title Overridden properties link todDefinition in Godot Docs Overridden properties link to Definition in Godot Docs Dec 8, 2020
@joshdegraw joshdegraw changed the title Overridden properties link to Definition in Godot Docs Overridden properties link to definition in Godot Docs Dec 8, 2020
@Calinou Calinou added this to the 4.0 milestone Dec 8, 2020
@joshdegraw
Copy link
Contributor Author

I had to take some time due to a house flood, but I am planning on finishing this PR ASAP.

author Josh DeGraw <joshmdegraw@gmail.com> 1607276564 -0700
committer Josh DeGraw <joshmdegraw@gmail.com> 1616823155 -0600

parent c6b9cea
author Josh DeGraw <joshmdegraw@gmail.com> 1607276564 -0700
committer Josh DeGraw <joshmdegraw@gmail.com> 1616822970 -0600

Add XML attribute to reference top level parent that defines overridden properties.

Modify makerst.py to use the new changes.

Add XML attribute to reference top level parent that defines overridden properties.

Sync classes reference template with current code base

Add inherits to PackedSceneGLTF.xml.

Modify makerst.py to use the new changes.

Modify makerst.py to use the new changes.

Fix clang formatting error.
@joshdegraw joshdegraw force-pushed the overrides-link-to-parent branch from f05c91c to 19d9bd3 Compare March 27, 2021 05:33
@joshdegraw
Copy link
Contributor Author

joshdegraw commented Mar 27, 2021

I kept the "overridden" property in the PropertyDoc struct in doc_data.h. Everything appears to be working. I looked for other things that utilize the overridden property and couldn't find anything. If we're happy with that being removed, I can go ahead and amend my commit to remove that property from the struct, otherwise this PR can be accepted as everything appears to be working.

@joshdegraw joshdegraw marked this pull request as ready for review March 28, 2021 02:18
@joshdegraw joshdegraw requested review from a team as code owners March 28, 2021 02:18
@joshdegraw
Copy link
Contributor Author

@Calinou Only asking since I've seen your eyes on this PR-- is there anything I need to do in order to get this merged?

@Calinou
Copy link
Member

Calinou commented May 23, 2021

@Calinou Only asking since I've seen your eyes on this PR-- is there anything I need to do in order to get this merged?

You need to rebase it against the latest master branch, as the PR now has merge conflicts.

@mhilbrunner
Copy link
Member

Bump :)

@YuriSizov
Copy link
Contributor

Superseded by #55559. Thanks for your contribution nonetheless! You've been attributed in the new PR for your efforts.

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.

4 participants