Skip to content

Missing spec tests #41

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 17 commits into from
Closed

Missing spec tests #41

wants to merge 17 commits into from

Conversation

jaredcnance
Copy link
Contributor

Closes #21
Closes #40

@jaredcnance
test(fetching): Request for an empty set should return an empty set
b93b9f7 
Issue #21 - 1
@jaredcnance
test(fetching): relationship request should ret null if unset
72a95ee 
Issue #21 - 2 
Relationship requests should return null if the relationship has not been set
@jaredcnance
feat(example-models): make owner optional on todo item
96b2efc
@jaredcnance
feat(example-models): db snapshot
e966f66
@jaredcnance
fix(*): allow null data objects
2b3aecb 
Issue #21 - 2
@jaredcnance
test(fetching): request for non-existant relationship ret 404
e0a9583 
Issue #21 - 3
@jaredcnance
refactor(Identifiable): there is no need to make Id abstract
18f6bac
@jaredcnance
example(*): add todo-item-collection
bbd2ccb
@jaredcnance
feat(identifiable): add default for generic
8b366d9
@jaredcnance
test(inclusion): multiple relationships can be included
8d28c80
@jaredcnance
test(inclusion): 400 response if relationship does not exist
1dd7845 
Issue #21 - 5
@jaredcnance
refactor(query-set): error details for nested rel
c729913
@jaredcnance
feat(*): add page manager to hold pagination details
cce3149
@jaredcnance
feat(root-links): add pagination links
e6d8c3f 
Issue #21
@jaredcnance jaredcnance mentioned this pull request Mar 1, 2017
Closed
@jaredcnance
Copy link
Contributor Author

Closed in favor of #42

Will deploy to master when P2 is completed

@jaredcnance jaredcnance closed this Mar 1, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@jaredcnance