scrable/CIS22C_FinalProject
Watch
1
0
Fork
You've already forked CIS22C_FinalProject
 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Master #3

Closed
weilinliu wants to merge 10 commits from master into master
pull from: master
merge into: scrable:master
Conversation 1 Commits 10 Files changed 4 +173 -4
weilinliu commented 2016-05-20 00:24:11 -07:00 (Migrated from github.com)
Copy link

simplified the setter functions

simplified the setter functions
scrable commented 2016-05-20 03:26:45 -07:00 (Migrated from github.com)
Copy link

I can't merge this. The phone number must be a double if we are allowing
area code. The max size of an int won't allow us to use area code.
On May 20, 2016 00:24, "williamL55" notifications@github.com wrote:

simplified the setter functions

You can view, comment on, or merge this pull request online at:

https://github.com/scrable/CIS22C_FinalProject/pull/3
Commit Summary

  • Merge remote-tracking branch 'scrable/master'
  • Merge remote-tracking branch 'scrable/master'
  • simplify setter functions

File Changes

Patch Links:

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/scrable/CIS22C_FinalProject/pull/3

I can't merge this. The phone number must be a double if we are allowing area code. The max size of an int won't allow us to use area code. On May 20, 2016 00:24, "williamL55" notifications@github.com wrote: > ## simplified the setter functions > > You can view, comment on, or merge this pull request online at: > > https://github.com/scrable/CIS22C_FinalProject/pull/3 > Commit Summary > - Merge remote-tracking branch 'scrable/master' > - Merge remote-tracking branch 'scrable/master' > - simplify setter functions > > File Changes > - _M_ Restaurant.cpp > https://github.com/scrable/CIS22C_FinalProject/pull/3/files#diff-0 > (33) > > Patch Links: > - https://github.com/scrable/CIS22C_FinalProject/pull/3.patch > - https://github.com/scrable/CIS22C_FinalProject/pull/3.diff > > — > You are receiving this because you are subscribed to this thread. > Reply to this email directly or view it on GitHub > https://github.com/scrable/CIS22C_FinalProject/pull/3

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bug

   
duplicate

   
enhancement

   
help wanted

   
invalid

   
question

   
wontfix
No labels
bug
duplicate
enhancement
help wanted
invalid
question
wontfix
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
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
scrable/CIS22C_FinalProject!3
No description provided.