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

Settings for more control over special deal prices #2037

Open
wants to merge 1 commit into
base: Dev
Choose a base branch
from

Conversation

fenhl
Copy link
Collaborator

@fenhl fenhl commented Jul 10, 2023

Proposing as an alternative to #1899. Instead of arbitrarily using the betavariate distribution for some price ranges and the uniform distribution for others, this PR adds settings to control the price range and distribution individually. As a bonus feature, I've also added an option to have each special deal location use the price of its vanilla item. Something like the features suggested in the comments of #1987 could be implemented as another option of this setting.

@cjohnson57
Copy link
Collaborator

I think this is the kind of thing I'd rather leave as a plando-only feature... This level of control over something so specific feels like GUI bloat

@cjohnson57 cjohnson57 added Status: Waiting for Author Changes or response requested Status: Under Consideration Developers are considering whether to accept or decline the feature described labels Nov 14, 2023
@fenhl
Copy link
Collaborator Author

fenhl commented Nov 14, 2023

I'm not sure if it's possible to add this functionality for plando while keeping the current interface for GUI settings. Might have to move this feature outside settings to achieve that. Will have to give it some more thought.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Setting specific to setting(s) Status: Under Consideration Developers are considering whether to accept or decline the feature described Status: Waiting for Author Changes or response requested Type: Enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants