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

Added an example using actix-web, async_graphql and diesel #538

Merged
merged 13 commits into from
Feb 28, 2024

Conversation

korir248
Copy link
Contributor

Pull Request

Related issue

Fixes #247

What does this PR do?

  • Adds an example which depivts using actix-web and async_graphql to query records in a postgres database

PR checklist

Please check if your PR fulfills the following requirements:

  • Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)?
  • Have you read the contributing guidelines?
  • Have you made sure that the title is accurate and descriptive of the changes?

Thank you so much for contributing to Meilisearch!

@korir248
Copy link
Contributor Author

korir248 commented Jan 3, 2024

@curquiza can you take a look

@irevoire
Copy link
Member

Hey, thanks, codewise the example is impressive. That’s awesome; I’m not familiar myself with diesel or graph ql, so having such an example in the repository is awesome!

But when I tried to cargo run --release the example, I got the following error:

= note: ld: library not found for -lpq
          clang: error: linker command failed with exit code 1 (use -v to see invocation)

Any idea why that would be the format?

Copy link
Member

@irevoire irevoire left a comment

Choose a reason for hiding this comment

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

Oh, and also, could you rename your Readme.md into README.md.

@korir248
Copy link
Contributor Author

korir248 commented Feb 27, 2024

Hey, thanks, codewise the example is impressive. That’s awesome; I’m not familiar myself with diesel or graph ql, so having such an example in the repository is awesome!

But when I tried to cargo run --release the example, I got the following error:

= note: ld: library not found for -lpq
          clang: error: linker command failed with exit code 1 (use -v to see invocation)

Any idea why that would be the format?

Yes. libpq-dev is required when working with Postgresql. sudo apt install libpq-dev should fix that error. Thank you

Copy link
Member

@irevoire irevoire left a comment

Choose a reason for hiding this comment

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

Ah yes perfect I see you updated the README as well thanks a lot!

bors merge

Copy link
Contributor

meili-bors bot commented Feb 28, 2024

Build succeeded:

@meili-bors meili-bors bot merged commit 1774889 into meilisearch:main Feb 28, 2024
7 checks passed
@curquiza curquiza added the documentation Improvements or additions to documentation label Mar 7, 2024
meili-bors bot added a commit that referenced this pull request Mar 11, 2024
545: Update version for the next release (v0.25.0) r=curquiza a=meili-bot

_This PR is auto-generated._

The automated script updates the version of meilisearch-rust to a new version: "v0.25.0"

---

Part of the work of #544

---

This version introduces features released on Meilisearch v1.7.0 🎉
Check out the changelog of [Meilisearch v1.7.0](https://github.com/meilisearch/meilisearch/releases/tag/v1.7.0) for more information on the changes. 

 ## ⚠️ Breaking changes

* `scoreDetails` feature is not experimental anymore. You can directly use `showRankingScoreDetails` during a search without activating the experimental feature 🎉

## 🚀 Enhancements

* Add proximity precision to settings (#548) cyprx

## ⚙️ Maintenance/misc

* Fix clippy (#546) curquiza
* Update dependencies (#542) omid
* Add an example using actix-web, async_graphql and diesel (#538) korir248


Co-authored-by: meili-bot <[email protected]>
Co-authored-by: Clémentine U. - curqui <[email protected]>
meili-bors bot added a commit that referenced this pull request Mar 11, 2024
545: Update version for the next release (v0.25.0) r=curquiza a=meili-bot

_This PR is auto-generated._

The automated script updates the version of meilisearch-rust to a new version: "v0.25.0"

---

Part of the work of #544

---

This version introduces features released on Meilisearch v1.7.0 🎉
Check out the changelog of [Meilisearch v1.7.0](https://github.com/meilisearch/meilisearch/releases/tag/v1.7.0) for more information on the changes. 

 ## ⚠️ Breaking changes

* `scoreDetails` feature is not experimental anymore. You can directly use `showRankingScoreDetails` during a search without activating the experimental feature 🎉

## 🚀 Enhancements

* Add proximity precision to settings (#548) cyprx

## ⚙️ Maintenance/misc

* Fix clippy (#546) curquiza
* Update dependencies (#542) omid
* Add an example using actix-web, async_graphql and diesel (#538) korir248


Co-authored-by: meili-bot <[email protected]>
Co-authored-by: Clémentine U. - curqui <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create more examples
3 participants