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

Changes the Keys Class #298

Merged
merged 3 commits into from
Mar 2, 2022
Merged

Changes the Keys Class #298

merged 3 commits into from
Mar 2, 2022

Conversation

alallema
Copy link
Contributor

Breaking Changes

The class Keys wasn't used as a proper class and did not return objects from the Keys but arrays. This has been changed. Setters and Getters have been added to the class.

  • client.getKey return a Keys
  • client.createKey return a Keys
  • client.updateKey return a Keys

Copy link
Member

@brunoocasali brunoocasali left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well done! 🥇

src/Endpoints/Keys.php Outdated Show resolved Hide resolved
src/Endpoints/Keys.php Outdated Show resolved Hide resolved
@alallema alallema marked this pull request as ready for review February 28, 2022 11:03
Copy link
Contributor

@bidoubiwa bidoubiwa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 🔥

@alallema
Copy link
Contributor Author

alallema commented Mar 2, 2022

bors merge

bors bot added a commit that referenced this pull request Mar 2, 2022
298: Changes the Keys Class r=alallema a=alallema

### Breaking Changes

The `class` `Keys` wasn't used as a proper class and did not return objects from the `Keys` but arrays. This has been changed. Setters and Getters have been added to the class.

- `client.getKey` return a Keys
- `client.createKey` return a Keys
- `client.updateKey` return a Keys

Co-authored-by: alallema <amelie@meilisearch.com>
@bors
Copy link
Contributor

bors bot commented Mar 2, 2022

Build failed:

@alallema alallema merged commit 9a8d04b into bump-meilisearch-v0.26.0 Mar 2, 2022
@alallema alallema deleted the changes-keys branch March 2, 2022 16:30
@brunoocasali
Copy link
Member

@alallema is there any particular reason to merge this into the bump instead of main?

@alallema alallema added the breaking-change The related changes are breaking for the users label Mar 9, 2022
bors bot added a commit that referenced this pull request Mar 14, 2022
292: Changes related to the next Meilisearch release (v0.26.0) r=alallema a=meili-bot

Related to this issue: meilisearch/integration-guides#181

This PR:
- gathers the changes related to the next Meilisearch release (v0.26.0) so that this package is ready when the official release is out.
- should pass the tests against the [latest pre-release of Meilisearch](https://github.com/meilisearch/meilisearch/releases).
- might eventually contain test failures until the Meilisearch v0.26.0 is out.

⚠️ This PR should NOT be merged until the next release of Meilisearch (v0.26.0) is out.

_This PR is auto-generated for the [pre-release week](https://github.com/meilisearch/integration-guides/blob/master/guides/pre-release-week.md) purpose._

Done:
- #296
- #298 
- #297
- #302 

Co-authored-by: meili-bot <74670311+meili-bot@users.noreply.github.com>
Co-authored-by: Amélie <alallema@users.noreply.github.com>
Co-authored-by: bors[bot] <26634292+bors[bot]@users.noreply.github.com>
Co-authored-by: alallema <amelie@meilisearch.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking-change The related changes are breaking for the users
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants