penguin-statistics / frontend-v2

The frontend of the Penguin Statistics website.
https://penguin-stats.io
MIT License
386 stars 55 forks source link

Adjust totals based on crafting #423

Open doubledusk opened 3 years ago

doubledusk commented 3 years ago

On the Farming Planner page, it would be nice if there was a feature that, once you gathered everything it told you to farm and got to the synthesis stage, it would update your totals based on your new mats crafted. For example:

  1. you need 2 Orirock Clusters
  2. you input that and click 'Calculate'
  3. system then tells you to go farm 1-7 for Orirock Cubes
  4. you do that
  5. once you have enough, you click on the 'Synthesis' tab in the 'Calculate' window
  6. it tells you to craft 2 Orirock Clusters
  7. (this is the suggestion part) you click on where it tells you to craft 2 Orirock Clusters (like a check box or something) to indicate you've crafted them
  8. system adjusts your totals by subtracting 10 Orirock Cubes (total to craft 2 Clusters) and adds 2 Clusters

Hopefully I explained that easily enough! Oh, just to be clear, I don't mean the random drops you get FROM crafting. Since that's random and I assume impossible to calculate. I just mean the things it tells you to craft specifically. Maybe it's just me but there have been a lot of times where I'm sitting on materials due to waiting on mastery training or a unit to come out, and after crafting something decide to work on something else while I wait. Which just turns in to a constant having to re-adjust my totals any time I craft something. Or maybe you gotta craft 30 Orirock Clusters and can only craft 15 before your Operator's become tired, and that would just help recalculating your totals while you craft in increments. It's not a big deal at all to be honest, I just thought it'd be a neat idea is all.

amenomori commented 3 years ago

Hi, and I would like to work on this issue for contribution.

Before we start completing this feature, some cases should be considered:

So I may take these below as assumed:

And I will also add this feature:

Now I have a try like this:

amenomori commented 3 years ago

462 is created for review before merging into dev branch.

GalvinGao commented 2 years ago

This would be fixed in the new v4 frontend, as a more feature-rich design of ArkPlanner will be implemented.