IronCoreLabs / IronCoreLabs/ironoxide

Add Display where it makes sense

Open
#175 0 comments 0 reactions 0 assignees View on GitHub
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.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.