Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

V1Meta object doesn't properly operate as list of assets #23

Open
mtalexan opened this issue Jun 26, 2018 · 1 comment
Open

V1Meta object doesn't properly operate as list of assets #23

mtalexan opened this issue Jun 26, 2018 · 1 comment
Labels
enhancement New feature or request

Comments

@mtalexan
Copy link
Owner

The memoized and custom constructed class is a V1Meta class object. This can potentially be a list if multiple results are returned from a query though. The support for more than one item being returned seems to be an update after the fact rather than initially supported by the design.

Split the V1Meta from an Asset class object so the query results can still be memoized by V1Meta, but a consistent interface to the asset objects can be provided wherein it's always a list of Asset objects that contain the attrs.

@mtalexan mtalexan added the enhancement New feature or request label Jun 26, 2018
@mtalexan mtalexan changed the title Memoized data and assets do not support a list of assets V1Meta object doesn't properly operate as list of assets Jun 26, 2018
@fahrenberger
Copy link

I agree. This has already bit me. If I request the TaggedWith property, V1Meta (in line 223) will only return the first tag of a Story/Epic. This is an actual issue that may cause people headaches. Maybe put a warning in the documentation?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants