AY1920S1-CS2103T-W11-2 / main

SplitWiser - Epic debt tracking
https://ay1920s1-cs2103t-w11-2.github.io/main/
MIT License
0 stars 5 forks source link

Add proper handling of empty email #204

Closed daekoon closed 4 years ago

daekoon commented 4 years ago

Context

We made a change in one of the previous PRs, that made the Email field optional when creating Contact objects. However, there was no proper support for handling empty email values, and those contacts without email specified were given the default value of "no@email.com" instead. This PR adds in a proper support of empty email fields using the Optional class.

Changes