Cedoor
Cedoor
@jimmychu0807 That PR is fine, thanks!
A better solution for this issue might be to use the original Blake implementation directly, wrapping it with a TS class.
@hannahredler sure, let us know if you'd like to work on this and we'll assign it to you.
Hi @Arch0125 , of course! I'll assign this issue to you
@Arch0125 are you going to work on the PR you opened: https://github.com/privacy-scaling-explorations/zk-kit/pull/361 btw, @thogiti I tagged you in a comment there
@semaphore-protocol/core I'm not sure if we should support this as well. Supporting TheGraph is already time-consuming. Could be a side-project separate from Semaphore tho. Wdyt?
@f3r10 sure! Is there enough information?
> @cedoor the only type supported for the message parameter should be `bitint` or `bigint | number | Buffer | Uint8Array`? `message` is limited to 32 bytes and is converted...
> Would be that ok? Yes, please make sure to update/add tests to check the correct type. `BigNumber` is either `bigint` or `string` (`string` must still be a stringified bigint).
Hey @Some-of-the-things, what documentation do you mean? https://github.com/privacy-scaling-explorations/zk-kit.rust/issues/9 is available as a good first issue again btw.