cowprotocol / cowprotocol/solana-programs
`Hash` type used by `OrderIntent.uid()` displays as base58
Nobody has claimed this yet.
- Dominant language
- Rust
- Stars
- 2
- Forks
- 1
- Avg merge
- 3d 1h
- Merged PRs (30d)
- 32
Description
For CoW orders, we typically use base16, so it would be best to wrap this type and set the dsiplay bindings tso they are proper by default. this was originally surfaced in the CLI, where a manual conversion is currently necessary https://github.com/cowprotocol/solana-programs/pull/26#discussion_r3574035672
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start at OrderIntent.uid() and inspect the Hash type's display bindings, using the CLI behavior and the linked discussion as context. The change is complete when the wrapped type displays CoW order hashes in base16 by default and the CLI no longer needs a manual conversion.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- blockchain
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 55/100