mtgred / netrunner

http://www.jinteki.net
Other
880 stars 387 forks source link

Some recent bugs I found #6537

Open maninthem00n opened 1 year ago

maninthem00n commented 1 year ago

Thanks for all you "coders of Jnet" do!

NoahTheDuke commented 1 year ago

Thanks for the report!

Ob is triggering on over install even thought

Can you explain in more detail? I don't understand.

maninthem00n commented 1 year ago

Yeah sorry I only half finshed that thought 😅

When I over install any assets, upgrade or ice, I'm getting an Ob prompt.

When I install an asset in a server with another assets the game prompts me with "the assets already there is being trashed" and then after I click OK I get an Ob trigger. Since Ob specifically says "not during installation" I assume this is a bug? Or just something that got missed.

Since trashing ice before installation is done manually there probably isn't a good fix for that specifically, but overinstalling isn't supposed to trigger ob, and I'm sure this will be confusing for other players.

Maybe whenever there is a manual trash there could be a pop-up asking if it's because of installation before the ob trigger?

Sorry for typos on my 3% battery phone. Hope that helps

NBKelly commented 1 year ago

Clyde triggering twice fits surprisingly well with his flavor text :joy:.

I know why ob does what it does, I'll try to figure out a good way to fix it some time soon.

francescopellegrini commented 1 year ago

Brain vs core damage was fixed by #6806