Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • Jul 16 21:49

    danielthepope on master

    Remove build script in favour o… (compare)

  • Jul 16 20:40

    dependabot[bot] on npm_and_yarn

    (compare)

  • Jul 16 20:40

    danielthepope on master

    Bump lodash from 4.17.15 to 4.1… (compare)

  • Jul 16 20:40
    danielthepope closed #59
  • Jul 16 06:53
    dependabot[bot] labeled #59
  • Jul 16 06:53
    dependabot[bot] opened #59
  • Jul 16 06:53

    dependabot[bot] on npm_and_yarn

    Bump lodash from 4.17.15 to 4.1… (compare)

  • Apr 25 23:44

    danielthepope on master

    Git history doesn't need to be … (compare)

  • Apr 25 23:36

    danielthepope on master

    Remove prestart step Dependency update Use Node 12, Alpine and Graphic… (compare)

  • Apr 25 23:31

    danielthepope on docker

    Remove prestart step Dependency update Use Node 12, Alpine and Graphic… and 1 more (compare)

  • Dec 30 2019 20:34

    danielthepope on no-phantom-trains

    (compare)

  • Dec 30 2019 20:33

    dependabot[bot] on npm_and_yarn

    (compare)

  • Dec 30 2019 20:33

    danielthepope on master

    Bump handlebars from 4.2.1 to 4… (compare)

  • Dec 30 2019 20:33
    danielthepope closed #58
  • Dec 30 2019 01:46
    dependabot[bot] labeled #58
  • Dec 30 2019 01:46
    dependabot[bot] opened #58
  • Dec 30 2019 01:46

    dependabot[bot] on npm_and_yarn

    Bump handlebars from 4.2.1 to 4… (compare)

  • Oct 06 2019 14:23
    danielthepope closed #53
  • Oct 06 2019 14:23

    danielthepope on master

    #53: Hide phantom trains from t… Fixes #53: Remove phantom train… (compare)

  • Oct 06 2019 14:20

    danielthepope on no-phantom-trains

    #53: Hide phantom trains from t… Fixes #53: Remove phantom train… (compare)

Daniel Pope
@danielthepope
Oh hey there Marcus :)
Marcus Noble
@AverageMarcus
:wave:
Daniel Pope
@danielthepope
I haven't even put the gitter badge on the readme yet!
Marcus Noble
@AverageMarcus
I haz skillz!
Marcus Noble
@AverageMarcus
I can't get it to work :(
Daniel Pope
@danielthepope
What's up with it?
Marcus Noble
@AverageMarcus
/media/shared/sync/Programming/GitHub/trntxt/node_modules/soap/lib/client.js:233
      result = obj.Body[output.$name];
                       ^
TypeError: Cannot read property 'GetDepartureBoardResponse' of undefined
    at /media/shared/sync/Programming/GitHub/trntxt/node_modules/soap/lib/client.js:233:24
    at Request._callback (/media/shared/sync/Programming/GitHub/trntxt/node_modules/soap/lib/http.js:60:7)
    at Request.self.callback (/media/shared/sync/Programming/GitHub/trntxt/node_modules/soap/node_modules/request/request.js:368:22)
    at Request.emit (events.js:110:17)
    at Request.<anonymous> (/media/shared/sync/Programming/GitHub/trntxt/node_modules/soap/node_modules/request/request.js:1219:14)
    at Request.emit (events.js:129:20)
    at IncomingMessage.<anonymous> (/media/shared/sync/Programming/GitHub/trntxt/node_modules/soap/node_modules/request/request.js:1167:12)
    at IncomingMessage.emit (events.js:129:20)
    at _stream_readable.js:908:16
    at process._tickCallback (node.js:355:11)
n/m
looks like my API key was wrong
Daniel Pope
@danielthepope
Ah, I was about to say check the API key :P
Marcus Noble
@AverageMarcus
my copy+paste skills suck!
Daniel Pope
@danielthepope
it should be in config/config.js
Marcus Noble
@AverageMarcus
yeah it was
Daniel Pope
@danielthepope
:thumbsup:
Marcus Noble
@AverageMarcus
but i must have missed a digit or something
nope, broke again :/
Daniel Pope
@danielthepope
I need a better way of checking stuff works. Some sort of test suite wouldn't go amiss ;)
Marcus Noble
@AverageMarcus
would be useful
Daniel Pope
@danielthepope
Hmm, what did you do to get that error?
Marcus Noble
@AverageMarcus
nothing
i think it must be something on their end
Boom! :sparkles: Pull request :exclamation:
Daniel Pope
@danielthepope

I had a similar error a few weeks ago: when you're calling the API you need to have an options object

var options = {
    numRows: 10,
    crs: 'RDG'
}

then pass that to GetDepartureBoard

Daniel Pope
@danielthepope
Just working out how to test a PR before I merge it. I might have brutally murdered the previous one...
Daniel Pope
@danielthepope
Added a couple of comments to your PR if you want to review those?
(I don't know why they didn't appear on the column on the right like in the JSOxford group - I've added the GitHub integration)
Daniel Pope
@danielthepope
Never mind, done it :)
Daniel Pope
@danielthepope
It's live :D :shipit: :confetti_ball: :sparkles:
Daniel Pope
@danielthepope
@AverageMarcus Did you ever manage to fix your error?
Marcus Noble
@AverageMarcus
Didn't get a chance to look
Daniel Pope
@danielthepope
Need regex help! #26
Marcus Noble
@AverageMarcus
Something like this? <a.*?>(.*?)<\/a>
Marcus Noble
@AverageMarcus
Ah hold on, missread it
Bens idea is totally better
Daniel Pope
@danielthepope
I think I did it
/<\/?((([^\/a> ]|a[^> ])[^>]*)|)>/ig
Erik Ros
@ejfrme
This message was deleted