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

feat: quest_shiny_probability #1094

Merged
merged 3 commits into from
Mar 13, 2025
Merged

feat: quest_shiny_probability #1094

merged 3 commits into from
Mar 13, 2025

Conversation

Mygod
Copy link
Collaborator

@Mygod Mygod commented Mar 12, 2025

image

Support for UnownHash/Golbat#267.

@jfberry @Fabio1988 I can't seem to find a quest with this field being nonzero though?

@Mygod Mygod added the Enhancement New feature or request label Mar 12, 2025
@Fabio1988
Copy link
Contributor

They might add and remove that toggle from time to time.... It was present for gotour :)
Maybe they just need to send it during some certain events?

@jfberry
Copy link

jfberry commented Mar 12, 2025

Maybe we should not include it when it is 0? If that is what it looks like when empty

@Mygod
Copy link
Collaborator Author

Mygod commented Mar 12, 2025

It is indeed annoying. It looks like there isn't a way to distinguish missing ShinyProbability versus it being zero in the proto?

@Mygod Mygod merged commit 33c51ad into develop Mar 13, 2025
2 checks passed
@Mygod Mygod deleted the shinyprob branch March 13, 2025 05:25
Copy link

🎉 This PR is included in version 1.36.0-develop.13 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@Mygod
Copy link
Collaborator Author

Mygod commented Mar 14, 2025

So i'm guessing that this is a shiny rate override option. If it's 0 then it's the global shiny rate that will be used instead.

@jfberry
Copy link

jfberry commented Mar 14, 2025

Or alternatively because they are paid for quests it may be ready to reveal odds under eu gambling laws

@Mygod
Copy link
Collaborator Author

Mygod commented Mar 14, 2025

Yes but they probably shouldn't only reveal it in a place that can only be seen through data mine?

@jfberry
Copy link

jfberry commented Mar 14, 2025

I’m thinking that this and other places (like eggs) which are related to purchases will start to comply with the law and show odds

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

Successfully merging this pull request may close these issues.

3 participants