Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Activity
  • Feb 22 00:55
    claassistantio commented #526
  • Feb 22 00:54
    dependabot-preview[bot] labeled #526
  • Feb 22 00:54
    dependabot-preview[bot] opened #526
  • Feb 22 00:54

    dependabot-preview[bot] on nuget

    Bump Microsoft.ApplicationInsig… (compare)

  • Feb 21 22:53
    claassistantio commented #525
  • Feb 21 22:53
    dependabot-preview[bot] labeled #525
  • Feb 21 22:53
    dependabot-preview[bot] opened #525
  • Feb 21 22:53

    dependabot-preview[bot] on nuget

    Bump Stripe.net from 34.24.0 to… (compare)

  • Feb 21 14:39
  • Feb 21 13:11
    ahmedsza commented #523
  • Feb 21 08:31

    dependabot-preview[bot] on nuget

    (compare)

  • Feb 21 08:31

    dependabot-preview[bot] on nuget

    (compare)

  • Feb 21 08:31

    dependabot-preview[bot] on nuget

    (compare)

  • Feb 21 08:31

    dependabot-preview[bot] on nuget

    (compare)

  • Feb 21 08:31

    dependabot-preview[bot] on nuget

    (compare)

  • Feb 21 08:31
    dependabot-preview[bot] closed #524
  • Feb 21 08:31
    dependabot-preview[bot] edited #524
  • Feb 21 08:31
    dependabot-preview[bot] closed #519
  • Feb 21 08:31
    dependabot-preview[bot] closed #518
  • Feb 21 08:31
    dependabot-preview[bot] closed #520
Frank Ebersoll
@frankebersoll
That's why I needed to fix those local references
Blake Niemyjski
@niemyjski
I was cousous about that. I like it (way more consistent) just never looked into why it wasn’t working in ts1.4
or maybe it was and I’m just a nub
Frank Ebersoll
@frankebersoll
I also like it how it is now.
Thought about changing everything from require to import.
Maybe I will put together a list of small refactorings and add them to an issue.
Finally got to reproduce the bug in tsproject, I opened an issue over there, too: ToddThomson/tsproject#45
Blake Niemyjski
@niemyjski
todd’s really helpful
I had so many issues and he was really great about helping
so our tests are just mocking submission now?
Blake Niemyjski
@niemyjski
I know i had it hitting a local server or returning invalid which didn’t test good when the server wasn’t running
Frank Ebersoll
@frankebersoll
yes, and we test the content now.
Blake Niemyjski
@niemyjski
wish we could fire up a server easily
wonder if we could point it at dev
Frank Ebersoll
@frankebersoll
hitting a local server would be an integration test - but I would rather set that up with a mocked server in Node.js, too.
Blake Niemyjski
@niemyjski
but better that we don't
yeah
someday I’d like to have integration tests
and auto deploy
Frank Ebersoll
@frankebersoll
:-)
Blake Niemyjski
@niemyjski
really like how you refactored this man
I like that we share this code
Frank Ebersoll
@frankebersoll
It wasn't a big thing, and it all came out of having to spawn that child process.
Seemed most logical like that.
Blake Niemyjski
@niemyjski
the only thing I see so far and we can do it later it’s 100% fine right now is using an interfaces for the callback definition
like in getSettings
var cb:interface = (interface) =>
Frank Ebersoll
@frankebersoll
like SubmissionCallback.ts?
Blake Niemyjski
@niemyjski
yeah
do you think it would be cleaner
I guess if we are only doing it in one or two spots it doesn’t make sense
Frank Ebersoll
@frankebersoll
i would do it if it turns out it's used often and has many arguments.
I'm okay with (a:int, b:int)=>int
Blake Niemyjski
@niemyjski
yeah
okay I think the only thing missing from the commit unless I missed it
is probably should update the express or node sample to throw the exception on startup
or do you think we should leave that out cause it would crash the app
hmm
yeah that would crash the app and then it wouldn’t work
so guess we don’t need it
commit looks great man!
I say commit it and we’ll do a round of testing and I’ll get my but in grear and fix a bug that I’m seeing and we’ll push it!!
Frank Ebersoll
@frankebersoll
okay! with "commit it", do you mean merge it? from then, I will work in a feature branch?
Blake Niemyjski
@niemyjski
merge it :)
Frank Ebersoll
@frankebersoll
alright. thanks for your review!
Blake Niemyjski
@niemyjski
what did you think of the code. feels so great having someone else doing major feature work on the code base
Frank Ebersoll
@frankebersoll
I'll do some final testing myself and then merge it later the day.
Blake Niemyjski
@niemyjski
if you ever see anything stupid, let me know.. I’ll never get offended just upset at myself for doing something stupid
sounds good :)
Frank Ebersoll
@frankebersoll
I haven't seen something stupid yet - have to say that I'm no JavaScript expert either.