Skip to content
This repository has been archived by the owner on May 15, 2024. It is now read-only.

Messaging API's #10

Closed
Redth opened this issue Feb 24, 2018 · 7 comments
Closed

Messaging API's #10

Redth opened this issue Feb 24, 2018 · 7 comments
Projects
v0.5.0-preview
Milestone
0.5.0-preview

Comments

@Redth
Copy link
Member

Redth commented Feb 24, 2018 •
edited by mattleibow

Issues moved to #25 (phone dialler), #26 (sms), #27 (email)
@Redth Redth added this to the Preview-1 milestone Feb 24, 2018
@mattleibow
Copy link
Contributor

Should these maybe be separate issues? Or are they all going to be in the same class? I think separate and small makes things easier: PhoneDialer, Sms, Email

@jamesmontemagno
Copy link
Collaborator

Agreed!

@Redth
Copy link
Member Author

Redth commented Feb 25, 2018

Not all the same class. We don't have to make an issue per class in general but we can split these ones out.

@mattleibow
Copy link
Contributor

PhoneDialler API's moved to #25.

@mattleibow
Copy link
Contributor

SMS API's moved to #26.

@mattleibow
Copy link
Contributor

Email API's moved to #27.

@mattleibow
Copy link
Contributor

Closing this issue as each API now has it's own issue.

@mattleibow mattleibow added this to Done in v0.5.0-preview Mar 28, 2018
@mattleibow mattleibow moved this from Done to Closed in v0.5.0-preview Apr 26, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
No open projects
v0.5.0-preview
  
Closed
Milestone
0.5.0-preview
Development

No branches or pull requests
3 participants
@Redth @mattleibow @jamesmontemagno