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

feat: [Validation] add support for dbGroup as parameter in is_unique and is_not_unique #9216

Open
wants to merge 9 commits into
base: 4.6
Choose a base branch
from
Open

feat: [Validation] add support for dbGroup as parameter in is_unique and is_not_unique #9216

wants to merge 9 commits into from
+113 −93

Conversation

maniaba
Copy link
Contributor

@maniaba maniaba commented Oct 6, 2024 •
edited
Loading

Description

See: #9215

Checklist:

  • Securely signed commits
  • Component(s) with PHPDoc blocks, only if necessary or adds value
  • Unit testing, with >80% coverage
  • User guide updated
  • Conforms to style guide

@ddevsr ddevsr added the wrong branch PRs sent to wrong branch label Oct 7, 2024
@ddevsr
Copy link
Collaborator

ddevsr commented Oct 7, 2024

This is enchantment, send to branch 4.6

@maniaba
feat: Add optional dbGroup support in validation rules for multiple d…
8907d03 
…atabase connections

cs fixer

docs format

fix rules

add test testIsNotUniqueWithDBConnectionAsParameter
@maniaba
docs: update v4.5.6.rst
f4f86b3
@maniaba
docs: fix tabs
325ffdc
@maniaba
docs update for v4.6.0
2fdc559
@maniaba maniaba changed the base branch from develop to 4.6 October 7, 2024 20:29
@maniaba
Copy link
Contributor Author

maniaba commented Oct 7, 2024

This is enchantment, send to branch 4.6

I’ve switched to branch 4.6, everything is now set. Could you please remove the 'wrong branch' label?

@ddevsr ddevsr added 4.6 enhancement PRs that improve existing functionalities and removed wrong branch PRs sent to wrong branch labels Oct 10, 2024
michalsn
michalsn reviewed Oct 11, 2024
View reviewed changes
Copy link
Member

@michalsn michalsn left a comment

Choose a reason for hiding this comment

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

The code looks good.

user_guide_src/source/libraries/validation.rst Outdated Show resolved Hide resolved
@maniaba maniaba requested a review from michalsn October 21, 2024 16:45
michalsn
michalsn requested changes Oct 21, 2024
View reviewed changes
Copy link
Member

@michalsn michalsn left a comment

Choose a reason for hiding this comment

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

The last things from me.

user_guide_src/source/changelogs/v4.6.0.rst Outdated Show resolved Hide resolved
system/Validation/StrictRules/Rules.php Outdated Show resolved Hide resolved
ddevsr
ddevsr reviewed Oct 22, 2024
View reviewed changes
system/Validation/Rules.php Outdated Show resolved Hide resolved
michalsn
michalsn approved these changes Nov 14, 2024
View reviewed changes
Copy link
Member

@michalsn michalsn left a comment

Choose a reason for hiding this comment

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

Looks good - thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ddevsr ddevsr ddevsr left review comments

@paulbalandan paulbalandan paulbalandan approved these changes

@michalsn michalsn michalsn approved these changes

@kenjis kenjis Awaiting requested review from kenjis

@samsonasik samsonasik Awaiting requested review from samsonasik

Assignees
No one assigned
Labels
4.6 enhancement PRs that improve existing functionalities
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@maniaba @ddevsr @michalsn @paulbalandan