These are chat archives for ecjs/Concierge

21st
Nov 2014
Vincent
@vlee90
Nov 21 2014 01:10
Got it in UTC time. There was an easy solution baked into xcode.
Mark Harrell
@ecjs
Nov 21 2014 01:13
Nice!
Let's test all of that tomorrow morning (make a call and stuff)
Just get it all ready for demo
Joe Elsey
@joeelsey
Nov 21 2014 01:32
hey. chareesa found a bug with one of the conceirge tests. its been fixed since.
Mark Harrell
@ecjs
Nov 21 2014 01:33
Ok
Joe Elsey
@joeelsey
Nov 21 2014 01:33
should i fetch the upstream? I saw a commit
Mark Harrell
@ecjs
Nov 21 2014 01:38
I am going to take a look now
not seeing any new pull requests on my end
Joe Elsey
@joeelsey
Nov 21 2014 01:41
I have not sent one
I'm only one commit ahead with that test fix
Mark Harrell
@ecjs
Nov 21 2014 01:41
oh ok..
I thought chareesa found a bug
Joe Elsey
@joeelsey
Nov 21 2014 01:41
She did. Its fixed and pushed to my dev. Do you want a pull request?
All tests are still green
I just upstreamed your one commit. All tests pass
Mark Harrell
@ecjs
Nov 21 2014 01:43
ok
yah
if you have a fix send it on over
Joe Elsey
@joeelsey
Nov 21 2014 01:43
Cool. Pull req incoming.
Mark Harrell
@ecjs
Nov 21 2014 01:45
merged
Joe Elsey
@joeelsey
Nov 21 2014 02:04
ok. I think I'm going to take off now.
Not that we currently have anything to worry about.
Mark Harrell
@ecjs
Nov 21 2014 02:23
LOL
Joe Elsey
@joeelsey
Nov 21 2014 02:23
He mentions sendgrid as well. What are the odds?
Mark Harrell
@ecjs
Nov 21 2014 02:24
haha
well listen..
this sounds a bit whiney to me
Joe Elsey
@joeelsey
Nov 21 2014 02:24
it is
Mark Harrell
@ecjs
Nov 21 2014 02:24
if tehy decided to turn off international sms
there is good reason
its a part of doing businss
business*
Joe Elsey
@joeelsey
Nov 21 2014 02:24
Probably because this guy was getting cheap international sms
Mark Harrell
@ecjs
Nov 21 2014 02:25
right..
he claims they are “screwing him over"
lol
Joe Elsey
@joeelsey
Nov 21 2014 02:25
Who knows what actually happened...
Mark Harrell
@ecjs
Nov 21 2014 02:25
he has a lot to learn
Joe Elsey
@joeelsey
Nov 21 2014 02:27
I was looking around for twilio docs and I found that
Chareesa Graham
@Chareesa
Nov 21 2014 02:37
Mark, I didn't touch the jobManager for clean up. Just letting you know. "Do you think we'll get dinged for not having tests on calls?" -Joe
signing off... home in.. like an 1.5-2hrs
Mark Harrell
@ecjs
Nov 21 2014 02:38
no..
I dont think we will get dinged
considering we havent learned sinon
and we cant make calls without an outbound endpoint available
Vincent
@vlee90
Nov 21 2014 02:41

Turns out we might need that route for getting the job information (date, recurring, etc) to populate the tableView of jobs. Right now, if a user creates a job, I have access to that information and can populate a tableView with it. However, if the user loads the application with their token, I can use GET /userInfo to get an Array of the job ids, but can't get the actual job information (date, recurring, etc) to populate the tableView.

It would be nice to have a tableView too see the job information that was saved on the sever, but this wouldn't be needed at all for the demo tomorrow.

Mark Harrell
@ecjs
Nov 21 2014 02:42
we have aq route for that
pretty sure
let me check
do a get request on /jobs
that will pull all available jobs for that user
Vincent
@vlee90
Nov 21 2014 02:49
oh nice! didn't see it
Mark Harrell
@ecjs
Nov 21 2014 02:49
;)
Vincent
@vlee90
Nov 21 2014 04:00
on the passwordReset route I seem to be getting a 503 error code back on both Postman and the application. Not a big issue for tomorrow's demo though.
Vincent
@vlee90
Nov 21 2014 04:10
On the PUT changePhone and PUT changeUsername, what is supposed to go into the HTTPBody? It won't seem to work if the body is empty.
Mark Harrell
@ecjs
Nov 21 2014 04:11
for username
its
for phone
its phone: “5555555555"
send them as objects like you have been doing
{“username”: “mark@mark.com”}
{“phone”:”3607295451”}
Vincent
@vlee90
Nov 21 2014 04:13
gotcha, I just realized there was i had two "//" in my routes on postman.
it works fine now ; )
Joe Elsey
@joeelsey
Nov 21 2014 04:51
I'm seeing if I can learn sinon tonight. Not likely, but I'm going to try.
Mark Harrell
@ecjs
Nov 21 2014 05:04
cool
I am trying to rebuild our call structure
it is…frustrating
Joe Elsey
@joeelsey
Nov 21 2014 05:08
I'm going very slowly and if I can figure it out I'm going to create a hello world tutorial for sinon because people like me need it.
Mark Harrell
@ecjs
Nov 21 2014 05:08
good point
Joe Elsey
@joeelsey
Nov 21 2014 16:05
38 commits last night
Nice.
Chareesa Graham
@Chareesa
Nov 21 2014 16:28
k, looking at jobManager for clean up
Chareesa Graham
@Chareesa
Nov 21 2014 17:18
error message "added Queued Job to Concierges Job array."
Joe Elsey
@joeelsey
Nov 21 2014 19:19
Make tests. Make tests modular. Run tests all of the time. Grunt test. Mocha test.