Skip to content
This repository has been archived by the owner on Oct 25, 2023. It is now read-only.

Added 'page' and 'per_page' parameters to 'search' - these are supported by Discogs API #54

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Blackout
Copy link

https://www.discogs.com/developers/#page:database,header:database-search
"Issue a search query to our database. This endpoint accepts pagination parameters"

Added 'page' and 'per_page' parameters to 'search' - these are supported by Discogs API

https://www.discogs.com/developers/#page:database,header:database-search
"Issue a search query to our database. This endpoint accepts pagination parameters"
@ricbra
Copy link
Owner

ricbra commented Jun 19, 2019

Hi @Blackout

Could you also add a unit test for this scenario?

@Blackout
Copy link
Author

@Blackout
Copy link
Author

@ricbra how do i make unit tests?

@froschdesign
Copy link

@Blackout
php-discogs-api uses PHPUnit. A complete documentation can be found under phpunit.de.
Look also in the tests folder of this repository.

@Blackout
Copy link
Author

Blackout commented Jun 20, 2019

Yeah no idea how to use PHPUnit. It works, you can see how simple the changes are. I'm using both changes now. Accept it or dont.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants