Added public initializer for KeyParams.

[mannuch]

Pull Request

Related issue

The KeyParams and KeysQuery structs do not have publicly accessible initializers. This implies that any code outside the MeiliSearch module cannot create new instances of these structs.

This works in direct contrast to examples laid out in the Meilisearch documentation such as when it displays constructing new KeyParams instances to pass to the POST /keys endpoint, which can be seen here, if you select Swift as the client language.

I've simply explicitly added a public initializers to both the KeyParams and KeysQuery structs.

[curquiza]

bors try

[curquiza]

Hello @mannuch
Thanks for the PR!
Looks like there is a linter issue. Could you fix it?

[meili-bors]

try

Build failed:

• linter-check

[brunoocasali]

Hi @mannuch thanks for your contribution!!

I would like to know if you have any tips to test this behavior, so we can prevent that from happening again in the future.

[brunoocasali]

Thanks a lot @mannuch! As @curquiza said, we still have some linting issues. Also, I posted a comment earlier asking about how we can test this :)

[brunoocasali]

Hi, @mannuch are you still interested in continuing working in this PR?

[mannuch]

Yes I am! Sorry I've been busy the past couple weeks, but I will try to implement the necessary changes at the end of the week.

Thanks for your patience!

[brunoocasali]

Thanks for contributing with Meilisearch ❤️

bors merge!

[meili-bors]

Build succeeded:

• integration-tests on macos-latest
• integration-tests on ubuntu-20.04
• linter-check

[brunoocasali]

Thanks again for contributing to Meilisearch ❤️
If you are participating in Hacktoberfest, and you would like to receive some gift from Meilisearch too, please complete this form.