IronCoreLabs / IronCoreLabs/ironoxide
Add Display where it makes sense
Open
tech debt
- Dominant language
- Rust
- Stars
- 12
- Forks
- 3
- Avg merge
- 5h 4m
- Merged PRs (30d)
- 3
Description
Places where we only have one basic implementation of Display, we should implement it. I was working with metadata results, and wanted to display the users a file was visible to. I expected to be able to:
```rs
metadata
.visible_to_users()
.join("\n")
```
but because `Display` wasn't implemented on something that is in reality `VisibleUser(UserId(String))`, I had to write:
```rs
metadata
.visible_to_users()
.iter()
.map(|u| u.id().id())
.collect::>()
.join("\n")
```
Contributor guide
No contributing guide indexed for this repository
Assessment
This issue has not been assessed yet.