computerpunc
computerpunc
@dnalborczyk This PR is ready for review. All 35 tests pass on AWS. Before merging from `master ` all unskipped tests passed locally but, for some reason, after merging with...
> @computerpunc just pulled down the PR as-is, and tests appear to run. is this branch based on `alpha 40`? > > ``` > Test Suites: 1 skipped, 23 passed,...
@dnalborczyk In any case, I found the bug: In `src/events/websocket/HttpServer.js` change `const routes = [...connectionsRoutes(this._webSocketClients), catchAllRoute]` to `const routes = [...connectionsRoutes(this._webSocketClients), catchAllRoute()]` Hope this helps and you can add the...
> @computerpunc just a quick update. applying the folder name changes locally and merge your PR-branch into master. If for some reason the tests don't pass we can fix it...