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.

Sign up for GitHub

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

Jump to bottom

Backport changes from doctolib fork #72

Open
wants to merge 149 commits into
base: master
Choose a base branch
from
Open

Backport changes from doctolib fork #72

wants to merge 149 commits into from

Conversation

pimpin
Copy link

@pimpin pimpin commented Jun 30, 2023

  • upgrade of Couchbase ruby client from 3.3.0 to 3.4.2
  • default scan consistency
  • remove of callbacks for nested documents
  • ignored_properties :
    • Change ignored properties signature for more similarity with AR ignored_columns
    • bug fix with #reload

CedricCouton and others added 30 commits February 28, 2023 17:03
@CedricCouton
can override scan consistency
59ecbf6
@CedricCouton
log scan consistency
6d4260e
@CedricCouton
add tests
c5167ce
@CedricCouton
fix usage with a thread local var for // execution
335ae29
@CedricCouton
fix usage with a thread local var for // execution
c037270
@CedricCouton
fix test
9a57ab3
@pimpin
use a DEFAULT_SCAN_CONSISTENCY const
8ac6625
@pimpin
allow to have nil value for scan_consistency
c620ef7
@CedricCouton
Merge pull request #14 from doctolib/can-override-scan-consitency
08ddfe0 
can override scan consistency
@pimpin
Prevent to expose dead callbacks for nested
7ef991f
@pimpin
Upgrade Couchbase ruby client version
d059dd2
@pimpin
Merge pull request #15 from doctolib/do_not_expose_some_callbacks_in_…
12500dc 
…nested_models

Prevent to expose dead callbacks for nested
@pimpin
Update ignored_properties api
013ec7b 
To be similar to (usual) ActiveRecord ignored column
@pimpin
Merge pull request #18 from doctolib/make_ignored_properties_similar_…
f44844c 
…to_ar_ignored_columns

Update ignored_properties api
@CedricCouton
test and solve cases of find with multiple ids
772e154
@CedricCouton @pimpin
Update lib/couchbase-orm/base.rb
856b5df 
Co-authored-by: Pierre Merlin <pimpin@users.noreply.github.com>
@CedricCouton
better lisibility
53b5f69
@CedricCouton
better lisibility
50f9af8
@CedricCouton
Merge pull request #19 from doctolib/fix-find-array
bdd4de1 
test and solve cases of find with multiple ids
@pimpin
Bump Couchbase ruby client to 3.4.2
02e5e08 
NB : 3.4.1 and 3.4.0 are not compatible with CouchbaseOrm
@pimpin
Merge pull request #16 from doctolib/upgrade-couchbase-ruby-client
31b5012 
Upgrade Couchbase ruby client version to 3.4.2
@pimpin
Add red test on strict_loading for instance
33a7d04
@pimpin
WIP first implem
51b50f4
@pimpin
Handle first, last and find
1280468
@pimpin
refacto test
ab1f125
@pimpin
consistant (and strange) implem
c022b29
@pimpin
fix
89ecb85
@pimpin
red again... implem is leaky
9379e9a
@pimpin
Fix leaky implem
79b54b9
@pimpin
handle habtm relations
571c92d
pimpin and others added 30 commits December 18, 2023 17:35
@pimpin
RM env variable
4b83f02
@pimpin
Fix require (unable to load)
0561811
@pimpin @JulienBURNET-FAUCHE
Raise on validation if schema is missing
cd3aee4 
Co-authored-by: Julien BURNET-FAUCHE <JulienBURNET-FAUCHE@users.noreply.github.com>
@JMGDoctolib
[BIC-1903] Add configuration for is null / is not valued
765bb15
@JMGDoctolib
Add readme
c846df5
@JMGDoctolib
Fix tests + Pierre review's
860db82
@JMGDoctolib
Update readme
78eb758
@JMGDoctolib
[BIC-1903] Add configuration for is null / is not valued
26ecf34
@JulienBURNET-FAUCHE
Merge remote-tracking branch 'origin/master' into BIC-1372
2d90e58
@JulienBURNET-FAUCHE
Merge pull request #28 from doctolib/BIC-1372
4a40a0e 
✨[Bic-1372] : Add Schema Validation on save actions
@tfe-github-1
Add .github/protected-branches.yml
4343bae
@tanguy-platsec
Delete .github/protected-branches.yml
3319a69 
This was created via Terraform apply but reverting the source PR did not delete the GitHub files...
@JMGDoctolib
[BIC-1903] Fix use of properties_always_exists_in_document for not ma…
3c40c35 
…tch expression
@pimpin
Merge pull request #35 from doctolib/BIC-193-FIX-NOT
31655d3 
[BIC-1903] Fix use of properties_always_exists_in_document for not match case
@JMGDoctolib
[APE-1598] Add the management of range in where clause
ab0cd54
@JMGDoctolib
Add exclude_end management
9906b72
@JMGDoctolib
Update readme for exclude_end
aa9b35e
@JMGDoctolib
Aurélien's review
6a9bb37
@JMGDoctolib
is_a? instead of instance_of?
ba623c9
@JMGDoctolib
[APE-1598] Add the management of range in where clause
2dd991b
@JulienBURNET-FAUCHE
⬆️ Upgrade couchbase driver
9da1e0a
@JulienBURNET-FAUCHE
Merge pull request #37 from doctolib/upgrade-cb-driver
dc964b9 
⬆️ Support couchbase > 3.4.2
@JMGDoctolib
[APE] Fix DATETIME format using serialize of DateTime
52bbed7
@JMGDoctolib
Manage Time and DateTime the same way using iso8601 and precision
6ec99b3
@pimpin
Merge pull request #38 from doctolib/APE-FIX-DATETIME
6bd5178 
[APE] Fix DATETIME format using serialize of DateTime
@clharge
handle multi criteria for nested document where clause
1f5f959
@pimpin
Apply suggestions from code review
f7ce316 
rm dev comments
@clharge
Merge pull request #40 from doctolib/multi-criteria-on-nested-documents
4f5b6e2 
Multi criteria for nested document where clause
@tfe-github-1
Add .github/workflows/check_pr_title.yml
543efce
@tfe-github-1
Update .github/workflows/check_pr_title.yml
f977aea
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@pimpin @CedricCouton @tanguy-platsec @cgenin-doctolib @beauraF @JMGDoctolib @JulienBURNET-FAUCHE @clharge