When adding t1i, I forgot to handle rifts initially. Everything looked OK, albeit harsh (since they always lead to OTC). It wasn't OK, but #1474 was the only way it could go wrong for now so it's all good. The other potential issue I spotted was that if your current timeline's max NP was larger than your max NP in OTC, you could use temporal rifts to get ahead in OTC. Still, I'll admit it would be a funny easter egg for those who somehow were able to use it. Also, for simplicity, the code is now using Math.ceil instead of Math.round, so the chance of jumping to your max NP is a bit higher.
eternaldensity
commented
7 years ago
When adding t1i, I forgot to handle rifts initially. Everything looked OK, albeit harsh (since they always lead to OTC). It wasn't OK, but #1474 was the only way it could go wrong for now so it's all good. The other potential issue I spotted was that if your current timeline's max NP was larger than your max NP in OTC, you could use temporal rifts to get ahead in OTC. Still, I'll admit it would be a funny easter egg for those who somehow were able to use it. Also, for simplicity, the code is now using
Math.ceilinstead ofMath.round, so the chance of jumping to your max NP is a bit higher.