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

Added self-delegation functions in Customer Entity #47

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

Conversation

shaibalster
Copy link

Addinging self delegation can help adding extra logic before we set the variable.
We can add the same extra logic in property set value too but it looks cleaner to have a separate function.
For example, before setting email, we can check the format.

Addinging self delegation can help adding extra logic
before we set the variable.
We can add the same extra logic in property set value too
but it looks cleaner to have a separate function.
For example, before setting email, we can check the format.
@shaibalster shaibalster marked this pull request as draft February 3, 2023 17:07
1 - Added Unique email rule violation exception catch and return null in case of violation
2 - Added email format validation rule checker
@shaibalster
Copy link
Author

Would really love to get review as I am from embedded system domain and learning internet-based applications development.

@shaibalster shaibalster marked this pull request as ready for review February 4, 2023 05:02
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.

2 participants