feat: enable consumption of unauthenticated notes

[mFragaBA]

addresses #418

Steps to test

• Until it get's merged, you'll need a node running on the polydez-unauth-notes branch
• Then, use the client as usual, but avoid syncing. Just try consuming notes as they are.

I tested a few flows consisting of a combination of:

• note type:
  • public
  • private
• moment of consumption
  • before syncing: this actually has 2 cases
  • when both the tx that creates a note and the tx that consumes it get included in the same block
  • when the tx that consumes the note gets included in a block after the one that creates the note
  • after syncing

TODO

• [X] allow consuming unauthenticated notes
• [X] ensure note records contain all info after consumption
• [x] add some integration tests

[mFragaBA]

• If the transaction involves any unauthenticated notes, their IDs should be included in the NoteId to input_notes: Option<NoteArgs>. If not, this should be an error (or should we include it automatically?).

This is already done so in the constructor of TransactionRequest

• We could also check that any notes that are not in the unauthenticated_input_notes list has an inclusion_proof in our local Store.

Would it be ok to do it in Client::new_transaction, before calling tx_executor.execute_transaction?

[igamigo]

Would it be ok to do it in Client::new_transaction, before calling tx_executor.execute_transaction?

Yeah, I think this would be the place to do it.

[Dominik1999]

Do you think we will get this into today's release?

[igamigo]

Do you think we will get this into today's release?

Yeah, this should make it in. It should be merged within the next few hours hopefully.