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

Add contact read/write contact info to Users #20

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

Conversation

ctgraham
Copy link
Contributor

Add work from @chryslovelace for User contact info with Addresses, Email, and Phone.

* add contactinfo and address classes and methods

* added a couple more address methods & tests

* added objects for Email and Phone

* add fixes for early exit case, @throws annotations

Co-authored-by: unknown <[email protected]>
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@ctgraham
Copy link
Contributor Author

ctgraham commented Dec 8, 2020

Ping @danmichaelo

@danmichaelo danmichaelo force-pushed the master branch 3 times, most recently from 05ac47a to dbb8592 Compare April 14, 2021 14:50
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

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.

3 participants