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

Invalid class imports for package elasticsearch/elasticsearch in matchory/elasticsearch v3 #375

Open
legacy-ua opened this issue Mar 14, 2024 · 4 comments
Labels
invalid This doesn't seem right

Comments

@legacy-ua
Copy link

In version 8.1 namespaces in this package https://github.com/elastic/elasticsearch-php had been changed, so when I tried to update laravel version I received next error: Class "Elasticsearch\ClientBuilder" not found, - because now it is located in Elastic\Elasticsearch\ClientBuilder. Old namespaces style is still used in elasticsearch/elasticsearch v7.17, but it is not compatible with matchory/elasticsearch v3.

Please, check this and fix.

@Radiergummi
Copy link
Member

v3 is not published yet; we didn't really get around to finish that version yet. Was there some issue with v2 for you?

@Radiergummi Radiergummi added the invalid This doesn't seem right label Mar 14, 2024
@legacy-ua
Copy link
Author

Thank you for your answer. Unfortunately v2 has next dependencies

"illuminate/pagination": "^9|^10"
"illuminate/support": "^9|^10"

that are not compatible with laravel 11. Is it possible to add ^11 to this declaration?

@Radiergummi
Copy link
Member

You're welcome. But... Laravel 11 has been released, like, two days ago 😄 We haven't even started migrating our own applications onto 11, so this is definitely going to take some time.

Thank you for your understanding! I'll close this issue once we're upgraded.

@legacy-ua
Copy link
Author

Thank you, just decided to notify you. Have a good day.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
invalid This doesn't seem right
Projects
None yet
Development

No branches or pull requests

2 participants