Resolve "Allow OR matching in addition to AND matching" #151

Merged
nik merged 3 commits from 29-allow-or-matching-in-addition-to-and-matching into master 2022-01-08 19:53:40 +01:00
nik commented 2022-01-07 15:07:51 +01:00 (Migrated from edugit.org)

Closes #29

Closes #29
nik commented 2022-01-07 15:07:56 +01:00 (Migrated from edugit.org)

assigned to @nik

assigned to @nik
nik commented 2022-01-07 15:14:04 +01:00 (Migrated from edugit.org)

added 1 commit

  • e6445cd4 - Register LDAPUserCreateRDNFields in site preferences

Compare with previous version

added 1 commit <ul><li>e6445cd4 - Register LDAPUserCreateRDNFields in site preferences</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-App-LDAP/-/merge_requests/94/diffs?diff_id=15718&start_sha=205758b4faecf920f7a43fdf81d4352cbcbcbe74)
nik commented 2022-01-07 16:08:38 +01:00 (Migrated from edugit.org)

added 1 commit

  • 3dcc09a9 - Allow combining matching fields for OR

Compare with previous version

added 1 commit <ul><li>3dcc09a9 - Allow combining matching fields for OR</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-App-LDAP/-/merge_requests/94/diffs?diff_id=15726&start_sha=e6445cd469cdae13b080d716e111fe567a4389ba)
nik commented 2022-01-07 16:14:31 +01:00 (Migrated from edugit.org)

assigned to @hansegucker and unassigned @nik

assigned to @hansegucker and unassigned @nik
nik commented 2022-01-07 16:14:34 +01:00 (Migrated from edugit.org)

marked this merge request as ready

marked this merge request as **ready**
nik commented 2022-01-07 16:15:17 +01:00 (Migrated from edugit.org)

added 1 commit

  • e4e57a19 - Allow combining matching fields for OR

Compare with previous version

added 1 commit <ul><li>e4e57a19 - Allow combining matching fields for OR</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-App-LDAP/-/merge_requests/94/diffs?diff_id=15728&start_sha=3dcc09a9cc81e67db2c9e43dd25d8c5b6ea7368d)
nik commented 2022-01-08 00:19:47 +01:00 (Migrated from edugit.org)

added 1 commit

  • f90827d1 - Allow combining matching fields for OR

Compare with previous version

added 1 commit <ul><li>f90827d1 - Allow combining matching fields for OR</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-App-LDAP/-/merge_requests/94/diffs?diff_id=15807&start_sha=e4e57a198d11f0b8fecfc5e635913622bc995e29)
hansegucker commented 2022-01-08 19:47:04 +01:00 (Migrated from edugit.org)
    choices = [("AND", _("All fields must match")), ("OR", _("Any one field must match"))]
```suggestion:-0+0 choices = [("AND", _("All fields must match")), ("OR", _("Any one field must match"))] ```
hansegucker commented 2022-01-08 19:53:16 +01:00 (Migrated from edugit.org)

resolved all threads

resolved all threads
hansegucker commented 2022-01-08 19:53:17 +01:00 (Migrated from edugit.org)

changed this line in version 5 of the diff

changed this line in [version 5 of the diff](/AlekSIS/official/AlekSIS-App-LDAP/-/merge_requests/94/diffs?diff_id=16024&start_sha=f90827d1f5d7c39d71bf9205367b222d9d8f06ce#fe3705841fefb154f44a0f3f02565766975b2d09_160_160)
hansegucker commented 2022-01-08 19:53:18 +01:00 (Migrated from edugit.org)

added 1 commit

  • 3539f9f9 - Fix choice definition for matching mode preference

Compare with previous version

added 1 commit <ul><li>3539f9f9 - Fix choice definition for matching mode preference</li></ul> [Compare with previous version](/AlekSIS/official/AlekSIS-App-LDAP/-/merge_requests/94/diffs?diff_id=16024&start_sha=f90827d1f5d7c39d71bf9205367b222d9d8f06ce)
hansegucker (Migrated from edugit.org) scheduled this pull request to auto merge when all checks succeed 2022-01-08 19:53:34 +01:00
hansegucker (Migrated from edugit.org) canceled auto merging this pull request when all checks succeed 2022-01-08 19:53:36 +01:00
hansegucker (Migrated from edugit.org) merged commit 37b061ce7a into master 2022-01-08 19:53:40 +01:00
hansegucker commented 2022-01-08 19:53:41 +01:00 (Migrated from edugit.org)

mentioned in commit 37b061ce7a

mentioned in commit 37b061ce7a4211dee6592a6d90e8c49425590d64
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
aleksis/AlekSIS-App-LDAP!151
No description provided.