-
Notifications
You must be signed in to change notification settings - Fork 34
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
Gold & Collectible Spawning #84
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This goes against the principles of integrating recovery features into a menu (specifically how YimMenu does it).
Though I really don't have any strong feelings for HorseMenu as I'm not really the person supposed to manage this.
oh yeah i should have expected this. who manages it? would it still be possible to merge even though it sort of goes against the principles? |
Who is supposed to manage this? This project is similar to YimMenu and has alot of potential however ik rdr2 is a bit dead. Id love to see at least a lua scripts addition to this menu like Yim has in it, that would open a ton of possibilities for myself and others tbh. |
Lua scripts is a big project 🤣 @Yimura I did however remember that red dead online is not actively developed so we would not need to worry about being targeted like YimMenu with recovery features. |
Download the artifacts for this pull request: |
there are merge conflicts for this? I am not at home so I can't see what they are |
alright conflicts have been resolved |
I am supposed to manage this :|. I have not decided on the recovery, depends on whether refraining from it on YimMenu has kept R* of our backs or not. Keep this open, ill discuss it with the yim team if they are interested to sound their opinions. |
Okay, I will keep this open. Idk if it helps but Red Dead Online isn't under active development anymore they only patch bugs so it would be improbable that they target us. |
If it helps, this drops sets of collectibles but you still have to pick them up 1 by 1 so it takes a bit, i tested it and compared to seeing another modder spawning treasure chests in the streets, this is a super light recovery method. Im down to make this like yim for gta, not in terms of styling, but functionality. Its actually helping me learn C++ as well and since it has only a handful of features, makes it easy to find/navigate where i need to be to add/change things |
I have decided not to enforce anti-recovery policy for this project. However, I will require all recovery options to employ a disclaimer. |
I will make this asap. |
@DayibBaba alright it has been updated, ready for merge? |
I appreciate how active you are and the contributions you are providing. But, to not risk overlooking small errors and bury them deep, I will from now on test PR's before merging. Currently installing Rdr2 to my device. EDIT: |
Is this a good new solution? |
Yes, however, I still want it in a group and maybe make the selectables activate on double click. Call the group Spawn Collectibles and remove all "Spawn all" prefixes. |
Okay, I will put it into a group called Spawn Colectibles. What dimensions do you think would be good for the group? Earlier, you discovered that the GetListBoxDimensions was not enough because of the large text. Do you think 1 or 2 columns should be created in the new "page" for recovery? |
Currently our Interface facilities are lack-luster. Use |
Alright. Do you think 2 columns should be good for now? |
Yes, those can be adjusted as needed. |
@DayibBaba alright this should be good. |
Allows rewards and collectibles to be spawned in game.
Thanks @tyackman and @Deadlineem for the help
closes #80