dev: make HasherT methods accept iterator rather than &[Felt] as argument

[tdelabro]

True for compute_hash_on_wrapper and compute_hash_on_elements.
Then check everywhere if it can allow us to remove some .collect() that were only there to respect the arg type.

Also pleases add a default impl for compute_hash_on_wrappers and make it call compute_hash_on_elements and remove the concrete impl of it for poseidon and pedersen.

[Gerson2102]

Can i work on this?

[tdelabro]

actually finish #1607 before claiming this one

[fishseabowl]

@tdelabro could I take this? Thanks

[tdelabro]

@fishseabowl can you rebase #1526 and #1610 before?

[fishseabowl]

@tdelabro #1526 and #1610 have been rebased. Please help to review them. Thanks

[tdelabro]

perfect! Go ahead

[github-actions]

There hasn't been any activity on this issue recently, and in order to prioritize active issues, it will be marked as stale.
Please make sure to update to the latest version and check if that solves the issue. Let us know if that works for you by leaving a 👍
Because this issue is marked as stale, it will be closed and locked in 7 days if no further activity occurs.
Thank you for your contributions!