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

[Feature Request]: Gameplay option to disable stash #7459

Closed
hfc2x opened this issue Oct 6, 2024 · 12 comments
Closed

[Feature Request]: Gameplay option to disable stash #7459

hfc2x opened this issue Oct 6, 2024 · 12 comments
Labels
enhancement New feature or request

Comments

@hfc2x
Copy link

hfc2x commented Oct 6, 2024

Feature Type

Quality of Life

Describe

I know you can just play the entire game without selecting Gillian's storage option, but it'd be a nice feature for people that just prefer her to be a conversation NPC only, like in vanilla. Also for people who don't want to go through the trouble of having to delete the stash file every new game.

@hfc2x hfc2x added the enhancement New feature or request label Oct 6, 2024
@Trihedraf
Copy link
Collaborator

This has been discussed before and shut down.

@kphoenix137 kphoenix137 closed this as not planned Won't fix, can't repro, duplicate, stale Oct 6, 2024
@AJenbo
Copy link
Member

AJenbo commented Oct 6, 2024

If you can't stop your self from using the storage, what's going to make you not just enable it?

@kphoenix137
Copy link
Collaborator

It's quite literally an optional and unobtrusive feature (evident by the fact we still have people asking how they can access stash). Simply making the personal decision to not use it is a lot less hassle than maintainers having to add yet another toggle/special case.

@hfc2x
Copy link
Author

hfc2x commented Oct 7, 2024

I concede that for items and weapons, it's alright to just not talk to Gillian. But the money being shared between characters is very obtrusive. Having two players using different characters that end up sharing their money (because it's visible to the shopkeepers) should not be a forced feature.

@AJenbo
Copy link
Member

AJenbo commented Oct 7, 2024

So your argument is that you sometimes want to use it for some characters, but not others?

@kphoenix137
Copy link
Collaborator

I do think it's a good point about the gold being used for towners. While it's convenient, I personally don't care for it, because I also like to have a degree of separation between characters, and shared gold breaks that barrier, which can be cumbersome in cases when I'm debugging (I need to rename the stash file to avoid accidentally spending legitimate gold)

@kphoenix137
Copy link
Collaborator

#7462

@hfc2x
Copy link
Author

hfc2x commented Oct 7, 2024

So your argument is that you sometimes one to use it for some characters, but not others

I don't know why you feel the need to respond with snark to a clarification about something a simple Google search will show that other people also consider an issue. Funnily enough, you also gave a snarky response to this person, as if moving/renaming stash.sv (in this person's case, on a device running Batocera) is not a cumbersome solution.

I do think it's a good point about the gold being used for towners. While it's convenient, I personally don't care for it, because I also like to have a degree of separation between characters, and shared gold breaks that barrier, which can be cumbersome in cases when I'm debugging (I need to rename the stash file to avoid accidentally spending legitimate gold)

#7462

Thank you for taking the time to consider this, and propose a solution.

@kphoenix137
Copy link
Collaborator

So your argument is that you sometimes one to use it for some characters, but not others

I don't know why you feel the need to respond with snark to a clarification about something a simple Google search will show that other people also consider an issue. Funnily enough, you also gave a snarky response to this person, as if moving/renaming stash.sv (in this person's case, on a device running Batocera) is not a cumbersome solution.

I do think it's a good point about the gold being used for towners. While it's convenient, I personally don't care for it, because I also like to have a degree of separation between characters, and shared gold breaks that barrier, which can be cumbersome in cases when I'm debugging (I need to rename the stash file to avoid accidentally spending legitimate gold)

#7462

Thank you for taking the time to consider this, and propose a solution.

I wouldn't perceive his responses as being "snarky". He is logical and matter-of-fact. His response to you was asking for clarification. Also, his native language isn't English, and he sometimes phrases things differently than a native English speaker that might be interpreted differently, especially without the nuance of vocal communication.

@AJenbo
Copy link
Member

AJenbo commented Oct 7, 2024

So your argument is that you sometimes one to use it for some characters, but not others

I don't know why you feel the need to respond with snark

I don't know why you think it's snark, I was just checking to see if I understood your argument correctly. I do have a tendency to be terse, maybe that's what you are picking up on. Additionally on GitHub I mostly use a neutral command tone as that's common when dealing with programming.

a simple Google search will show that other people #5278.

Why should I do a google search to find a conversation I remember?

as if moving/renaming stash.sv (in this person's case, on a device running Batocera) is not a cumbersome solution.

I gave them an answer they can benefit from now rather then having to wait for a solution to be programmed and released, your talking as if that is a simple thing to do.

@hfc2x
Copy link
Author

hfc2x commented Oct 8, 2024

I gave them an answer they can benefit from now rather then having to wait for a solution to be programmed and released, your talking as if that is a simple thing to do.

I see. In that case, I want to ask: would it be okay for me to make a pull request for this then? Just by glancing at the code, I think I can add an option hide the dialogue option and the gold stored in the stash from being visible to shopkeepers. I'm obviously not a maintainer, so I don't know the gameplay ramifications of doing this, so this approach could be too simplistic.

I'm talking about adding a gameplay option that wouldn't interfere with the stash code in any way, just optionally hiding it from the player, and I already have an idea of how to do it (obviously for QoL reasons, the stash should be enabled by default, like it is right now).

@AJenbo
Copy link
Member

AJenbo commented Oct 8, 2024

We are considering some other options, there's a bit more context here: #7462

Ideally options likes these should become mods that you can toggle, that's the goal of the next major version. If you can contribute towards that it would be very welcome.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants