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

Tests/basic functionality #10

Closed
wants to merge 14 commits into from
Closed

Tests/basic functionality #10

wants to merge 14 commits into from

Conversation

mechelon
Copy link
Member

Internal PR

johguentner and others added 11 commits May 16, 2021 13:52
implement collectChildren of EntityCollection as general method for collecting diverse entities at the same time (needed for search).
currently implemented for object-types 'page' and 'database'.
pull current state of main
add prototypical search-endpoint, allow general EntityCollections
implement simple filter and sort for search-queries
g and added databases endpoint tests
@mechelon mechelon requested a review from johguentner May 18, 2021 22:04
@johguentner
Copy link
Member

😎 looks great!
thanks @mechelon for the headsup!

just found one issue on my side:
The get-method within Endpoint doesn't return the response, which leads to issues within certain Entities (for example if I want to retrieve a specific block).
Can you double check that @mechelon ? 😊

put missing return of response within get-method of Endpoint
@johguentner
Copy link
Member

with the small fix above, this should work now as expected

@mechelon
Copy link
Member Author

😎 looks great!
thanks @mechelon for the headsup!

just found one issue on my side:
The get-method within Endpoint doesn't return the response, which leads to issues within certain Entities (for example if I want to retrieve a specific block).
Can you double check that @mechelon ? 😊

No, this was on purpose :) One of the things I want to discuss with you later. Should've marked it, sorry.

@mechelon mechelon closed this May 19, 2021
@mechelon mechelon deleted the tests/basic-functionality branch May 29, 2021 11:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants