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

Revise Spectral Usability #7662

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

kphoenix137
Copy link
Collaborator

@kphoenix137 kphoenix137 commented Jan 17, 2025

Allows Spectral Elixir to become permanently usable when completing the mushroom quest. Previous, the usability of the elixir is based on quest completion on a game to game basis.

@FitzRoyX
Copy link

This creates some awkwardness, because Pepin told you to deliver it to Adria. That's why it unusable, it's not yours yet. So they'd both know you stole it, but there's no dialog against that action, so it just silently completes the quest? You'll also get bug reports, because the reasons for the change aren't obvious. "Well you see the item has no drinkable flag, so when you stash it away, it goes back to being unusable in the next game, and devx won't change the save format, blah blah blah". Yeah not going to work.

Not saying it's not a small issue, but the only proper way to fix it along with 100 other things is to move on from the vanilla save format at some version point.

@StephenCWills
Copy link
Member

This creates some awkwardness...

No awkwardness was created. You're just misunderstanding how this PR works.

... so it just silently completes the quest?

At the point where you complete the quest, the elixir becomes permanently usable. If you stash it at that point, you can take it back out and use it in any game session regardless of whether you've completed the quest in that game session.

If you stash it without completing the quest, you can only use it in game sessions where the quest is already completed. Essentially, it would retain the awkwardness it already has.

@kphoenix137
Copy link
Collaborator Author

I revised the description

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

Successfully merging this pull request may close these issues.

4 participants