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

Debug system #490

Open
wants to merge 13 commits into
base: main
Choose a base branch
from

Conversation

HarshNarayanJha
Copy link
Contributor

Do you have a forum thread linked to this PR?
Yes, sure: https://forum.unity.com/threads/debug-system.1151783/

What will this PR bring to the project for everyone?
A Debugging System which will certainly help everyone developing the project.

Why are these changes necessary?
If we want to go to another location while in play mode, we have to either restart or walk all the way. With this Debug System, we can warp to any location quickly. More info in thread.

How did you implement them?
See in thread for full implementation.

Also @ciro-unity, pull request can't be automatically merged, but don't worry, you can accept both changes when it conflicts. Only one line that I have deleted is this:
Screenshot_20210820-081059_Chrome

This is the localization entry I did, so accept my change in that place.
Hope this helps.

Thanks.

Added DebugMenu prefab.
Added some checks and variables to check if this is really debug mode (or editor mode) before performing any debug operations. EditorColdStartup sets the isDebugMode bool to true.
@HarshNarayanJha
Copy link
Contributor Author

Hey Ciro, no need to fix conflicts. I fixed all of them when I realized that I am able to do so here.

@ciro-unity ciro-unity added the tool A PR that contains an Editor tool or custom Inspectors label Sep 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tool A PR that contains an Editor tool or custom Inspectors
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants