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

Support Sharing calculation #587

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

Conversation

ForwardFeed
Copy link
Contributor

@ForwardFeed ForwardFeed commented Nov 5, 2023

in reference to #499
I haven't fully tested it however as I don't know too much how should I test it.
in todofix there is this bug with Dynamax and non-full HP situation.

@AustinXII
Copy link
Member

@thejetou opinions? I'm sure a lot of people would like this feature.

@AustinXII
Copy link
Member

After some testing, it does seem to works for abomasnow but fails to work if you try sharing a calc with a different pokemon. I'll see if I can figure out whats going on and hopefully we can merge this.

@AustinXII
Copy link
Member

Oh I see, it only doesn't work if the Pokémon has more than one form

@AustinXII
Copy link
Member

Also the set should only be shown in the same generation the damage calc was made in. Currently if you shared a damage calc and the person uses the link, that set would be shown in all generations.

@ForwardFeed
Copy link
Contributor Author

ForwardFeed commented Jul 2, 2024

When I did the code a while back ago it was supposed to redirect the receiver gen to the sender gen. this PR is likely not up to date of the current codebase on many points.

@AustinXII
Copy link
Member

@ForwardFeed I would love to add this if you’d take a jab at updating this PR now that I’m back.

@ForwardFeed
Copy link
Contributor Author

ForwardFeed commented Jul 3, 2024

I'm sorry I'll probably not work on the smogon calc in the future. I've made this PR 9 months ago, In between I have made my own fork where i changed a lot of code that i don't remember well the codebase of the smogon calc and currently If I'm again working on a calc I would want to make my own nuzlocke specialized calc would I have time and value for that.

@AustinXII
Copy link
Member

@ForwardFeed no problem at all. We appreciate the PR, I’m sorry for the ghosting and no response.

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.

2 participants