koopjs / koop-provider-agol

ArcGIS Online provider for Koop (public services only).
Other
12 stars 10 forks source link

Major refactor of the entire app + switch to node-resque #79

Closed dmfenton closed 8 years ago

dmfenton commented 8 years ago

Added

ungoldman commented 8 years ago

y'all got some rebasin' to do

dmfenton commented 8 years ago

Rebased :)

ungoldman commented 8 years ago

There's a whole lot going on in here (~11000 lines added/deleted) in one big commit. Should I just merge and hope for the best?

chelm commented 8 years ago

This should be converted to es2015 first

ungoldman commented 8 years ago

:neutral_face:

dmfenton commented 8 years ago

Not sure how I could have split this up, everything is kinda interrelated. Given that its been through pretty substantial testing locally and is going to go through a lot of QA before release I think we'll be pretty safe when we release it.

That said, I would love some feedback on the way I structured this thing.

@chelm ha, I think I'll save that for after the honeymoon.

dmfenton commented 8 years ago

I'm going merge this and publish a pre-release. That will make it easier for us to take this through our formal QA. We can still incorporate any bug fixes and feedback before a final release.

ungoldman commented 8 years ago

:+1: