fix: assume unknown tx type is TransactionType.Legacy

[jaketimothy]

Fixes #3489
Fixes #3458

[jaketimothy]

Let me know if this is too permissive

[MicaiahReid]

I think I'm cool with this. Thanks for the PR!

I would like to note - this should fix your problems with Arbitrum Nitro throwing with an "invalid transaction types" error, but we still won't actually support these transaction types. Or any non EVM compatible chains. (I know, Arbitrum claims to be EVM compatible, then they go and start making up transaction types! 😅 )

We are working on a plugin system so that users can extend Ganache to work with different L2s. Keep an eye on our releases for when this will be available. Thanks again!

[davidmurdoch]

Awesome! Thanks for doing the work for this. I just have one change request.

[davidmurdoch]

Thanks @jaketimothy for those quick test updates. Tests are running now and we'll get this approved and merged soon!

[jaketimothy]

@davidmurdoch lmk if there's anything more I can take care of here!

[davidmurdoch]

Looks perfect! Thanks!