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

feat: add lazy evaluation to criteria callable - Craft 3 #269

Closed
wants to merge 50 commits into from

Conversation

qrazi
Copy link

@qrazi qrazi commented Aug 19, 2023

Refers to #257

janhenckens and others added 30 commits January 18, 2022 20:27
Signed-off-by: Jan Henckens <jan@henckens.be>
Signed-off-by: Jan Henckens <jan@henckens.be>
Signed-off-by: Jan Henckens <jan@henckens.be>

# Conflicts:
#	CHANGELOG.md
#	composer.json
#	src/console/controllers/scout/IndexController.php
#	src/models/Settings.php
Signed-off-by: Jan Henckens <jan@henckens.be>
Signed-off-by: Jan Henckens <jan@statik.be>
Signed-off-by: Jan Henckens <jan@statik.be>
Signed-off-by: Jan Henckens <jan@henckens.be>
janhenckens and others added 20 commits June 5, 2023 16:54
3.1.1

# -----BEGIN PGP SIGNATURE-----
#
# iQGzBAABCAAdFiEE8kTipvHyprdyGl56wLsWSOmqbZMFAmSCBCgACgkQwLsWSOmq
# bZM67gwAmiuLJCFbTdtafsWceSlSqxOWEcVv8zsTaCfYaHrlenNo/u3IbIdEXxuD
# X8OqXZO1dYEeTgSKmQTRMWlFxgAgvQl0vX6F5pAW6g5cd+tUzJ9v7fFUPWrTTELA
# 7xOdtvFt2jGpeWlBv2ROr/ystSqxVDeZq2Vhl59Mg1qaVnKp0En7xlrs4+ohDoaj
# l/gG5mgFRNtPHKajU7iHdgZVW814vI4VtiFmOGFUeldpTVr6PPY6YDjlcNWfRTNG
# qB8N6RAKR8V01y+CXNPMA2Ci04vi1SHbO/8ayxZbnY8SRhOm/lXS/7bSdm5MPaZZ
# osNJ61gRRRygxvibQMckMTnDtS1vwfhH5QjznOuK6xNhbT3YoJbg+ok2ZvNU1H60
# 07O2krcIlulTnF5G9HifigydvkCuf0O5BCvK7pknAz8fmgL034rbeNqU7QqrXi0E
# SmsvgX9FN/X8LLYBJrgxExdndMXlRdbm7dG7PV/UxcF943058DvvNiDIv9A4AuiI
# d4AiYocX
# =IVyc
# -----END PGP SIGNATURE-----
# gpg: Signature made Thu Jun  8 18:39:04 2023 CEST
# gpg:                using RSA key F244E2A6F1F2A6B7721A5E7AC0BB1648E9AA6D93
# gpg: Good signature from "Jan Henckens <jan@henckens.be>" [ultimate]
- Add required property when creating Field (EventHandlersTest, SearchableBehaviorTest)
- Mark failing test as incomplete for now (EventHandlersTest)
- Set config value to test environment (ScoutUtilityTest.php:it_returns_html)
- Pass Queue instance from Application instead of new instance to prevent error "Queue must be an application component." (ImportIndexTest.php:it_calls_update_on_the_engine)

Refers to studioespresso#226
…DB user as expected by test config and GH Actions

Refers to studioespresso#226
Fix tests for Craft 4-version of the plugin
@qrazi qrazi force-pushed the feat_lazy_evaluation_craft3 branch from 9650f0d to 166de11 Compare August 27, 2023 12:32
janhenckens added a commit that referenced this pull request Aug 27, 2023
@janhenckens
Copy link
Member

I manually added these changes to the Craft 3 branch and they are released in the latest beta version:

composer require studioespresso/craft-scout:2.8.0-beta.3

@qrazi qrazi deleted the feat_lazy_evaluation_craft3 branch January 8, 2024 12:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants