Base resource struct #37
Conversation
Note: the fixtures are tweaked so we can test that HAL links are preferred before fields.
| } | ||
| return r.rels | ||
| return r.Resource.RelsOf(r) |
There was a problem hiding this comment.
I would love to not have to implement this on every type. But the *Resource has no knowledge of the actual API resource, so it can't access the hypermedia fields.
There was a problem hiding this comment.
I think I have a solution to this. What we are trying to do is to fill the hypermedia.Relations map either from the GitHub xxx_url convention or from the HAL convention link convention. Instead of overriding Rels(), we could inject hypermedia.Relations after the http call. We could use reflection to inject the relations map. I'll create a PR to further demonstrate.
There was a problem hiding this comment.
That probably works better. No reason this has to be a function on the type.
|
Gonna close this as it looks stale. Please re-open if I'm mistaken. |
Add's a base Resource type to Root and PullRequest.