These are chat archives for wheres-my-receipt/give_me_your_money

27th
Mar 2015
Sarah Johnston
@sarahabimay
Mar 27 2015 00:24
how do i get authorized to send payments? the true/false then submit route doesn't work
James Griffiths
@MIJOTHY
Mar 27 2015 00:24
which payments
Sarah Johnston
@sarahabimay
Mar 27 2015 00:24
either annual or desk
James Griffiths
@MIJOTHY
Mar 27 2015 00:25
annual you should be able to pay from the get-go
desk the true-submit should work
lemme give it a try, sec
what message are you getting back?
Sarah Johnston
@sarahabimay
Mar 27 2015 00:26
as you say, annual works
James Griffiths
@MIJOTHY
Mar 27 2015 00:27
select true, hit submit, then go to api/accounts/youraccountname
and check what 'desk_authorization_status' is
Sarah Johnston
@sarahabimay
Mar 27 2015 00:27
you aren't authorized to do that yet or some such thang
i'm not working from your latest code
James Griffiths
@MIJOTHY
Mar 27 2015 00:27
yeah the old one should work
latest doesn thave the true/false button
Sarah Johnston
@sarahabimay
Mar 27 2015 00:56
my pull request is in
James Griffiths
@MIJOTHY
Mar 27 2015 01:01
you have some >>>>>development things left
l116 account l33signup
i'll pull, merge with mine, fix up stuff and push to dev
Sarah Johnston
@sarahabimay
Mar 27 2015 01:03
Ah really sorry. Tired eyes! Nighy
James Griffiths
@MIJOTHY
Mar 27 2015 01:07
pushed to dev with latest version (sarah + mine changes)
i took the liberty of putting a cheesy gold border around the profile pic
or glow, rather
James Griffiths
@MIJOTHY
Mar 27 2015 01:18
re-did it, accidently pushed up the version i had just pulled instead of my merged version
James Griffiths
@MIJOTHY
Mar 27 2015 09:16
on the train, i will start implementing moment in the accounts page for date formatting, try to do some more formatting, and get started on constructing some mocks for the test fest that may or may not happen today
ill push up my latest to branch - only minor changes. including my last push, ive fixed the stripe membership button, made it so that the client can edit account details for people other than myself (i.e. themselves instead), made an alert wrapper for the alerts (so they hang right, aren't obstructed by the navbar and dont screw with the formatting of the rest of the page), and made the golden profile glow nicer
Sarah Johnston
@sarahabimay
Mar 27 2015 09:20
okay cool. I'm just working through the flow and making sure we get the pages we should when we are or aren't logged in. I'm also going to fix the display of alerts now the styling has changed. I should try and get some tests written too. did we move away from lab in the end?
okay grand, you've fixed the alert issue so I won't need to now.
James Griffiths
@MIJOTHY
Mar 27 2015 09:20
i think lab will be fine, local db seems the way to go
Sarah Johnston
@sarahabimay
Mar 27 2015 09:21
will get you to explain that later
James Griffiths
@MIJOTHY
Mar 27 2015 09:21
ok check __server.js
at the start of the api tests, there is a connection to mongoose and it just connects to your local DB
(mongod stuff)
Sarah Johnston
@sarahabimay
Mar 27 2015 09:21
will do. also, you'll see I put in the payment receipt email, can you check that's working for you? I'll also try and use a different sender
so the stop going to spam
cool on the db stuff. would that have fixecd your issue last week witht the handing db
James Griffiths
@MIJOTHY
Mar 27 2015 09:22
i think perhaps it would have
oh wow i cant push, its just hanging on me
Sarah Johnston
@sarahabimay
Mar 27 2015 09:23
noooo
James Griffiths
@MIJOTHY
Mar 27 2015 09:23
think its time for a restart
see you at the office m80s
Sarah Johnston
@sarahabimay
Mar 27 2015 09:24
okay, i'll be in about 11:30 but will be working till then so you can gitter me. though i may have a short kip if you don't get me..
Sarah Johnston
@sarahabimay
Mar 27 2015 09:50
created a pull request, please merge
this sorts the divs on the account page but need to sort out the divs on the admin pages too
Adam Kowalczyk
@adamkowalczyk
Mar 27 2015 10:10
merged
guys, can we merge dev to master for review? will also update read me
Sarah Johnston
@sarahabimay
Mar 27 2015 10:26
I was thinking the same on both accounts.
Adam Kowalczyk
@adamkowalczyk
Mar 27 2015 10:27
dan has started having a look. If there are no objection, I’ll merge dev to master now (assuming it runs, will double check!)
Sarah Johnston
@sarahabimay
Mar 27 2015 10:30
It's not the finished product but should work. Admin pages look shit as that's what I was going to tackle this morning
Adam Kowalczyk
@adamkowalczyk
Mar 27 2015 10:30
no worries, doesn’t need to be finished to be on master, just working. We should ahve kept it more uptodate i suppose, but meh
although, seem to be having trouble getting hithub to accept a push. think others are to, so github issue
Sarah Johnston
@sarahabimay
Mar 27 2015 10:32
Yes I did have an issue. James too
Sarah Johnston
@sarahabimay
Mar 27 2015 20:42
made some changes. pull request waiting!