-
Notifications
You must be signed in to change notification settings - Fork 113
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
Up to date fork UE5, UE4.27+ #20
Comments
What doesn't work? |
Any updates recently? How does this compare to https://github.com/Takexito/DonAINavigationUE5 ? It seems to have some of your commits in it. |
Project has spread out into many forks. You have to change more than just the Engine Version though as you can see from the commits. I haven't checked this other fork you linked, but on a preliminary glance it seems to be mostly the same. I will check if I should port back some changes from there as soon as I have time. |
Yeah it seems to be almost identical. I had to make a change to one line to get it working with 4.27. I maintain 2 forks with a number of other fixes/improvements. The one with the most changes was made for a client project though so I can't share those without permission. At some point I'll see if I can add those improvements to my public fork and report back. As a side note, I'm pretty sure that if you remove the engine version line from the uplugin file it will be treated as engine version agnostic. You may need to run a command to package it but all you have to do is drag and drop most of the time (at least for C++ projects). See this plugin for example: https://github.com/AutoSizeComments/AutoSizeComments/ |
In the last year I have collected fixes from various forks and added some of my own. I also updated the plugin for UE5 and UE4.26, I think this repository might have been abandoned, but @VSZue: If you ever see my PR or this Issue consider merging it.
In case this repository is really abandoned, Issues & PRs in my fork are welcome, I sometimes check other forks for new developments too:
master...FAUSheppy:master
PR: #19
The text was updated successfully, but these errors were encountered: