These are chat archives for systemaccounting/mxfactorial

7th
Aug 2016
Mike Klishevich
@klishevich
Aug 07 2016 10:38
I created my Firebase database, according to instructions (additionally I changed .firebaserc file to set default project to my Firebase), no errors, but in firebase console database is empty (mxfactorial-test:
null). Maybe I need to do some database migrations? I started server npm run dev and tried to change email in Account settings again. I got the same error: after start local server http://localhost:8080/socket.io/?EIO=3&transport=polling&t=LPaX7ho , and while saving new email error xhr.js:161 PATCH http://localhost:8080/systemaccounting/account/email 404 (Not Found)
Mike Klishevich
@klishevich
Aug 07 2016 10:47
Other parameters I set:
export const BASE_URL = 'http://localhost:8080/systemaccounting'; export const SOCKET_URL = 'http://localhost:8080/';
auth-hooks.js file tried comment out requireAuth, and uncomment
FIREBASE_URL: 'https://mxfactorial-test.firebaseio.com', FIREBASE_KEY_PATH: 'mxfactorial-test-edb14399d2e8.json', (mxfactorial-test my Firebase)
Max Funk
@mxfactorial
Aug 07 2016 14:52
Wait for @hotay since socket.io
... was added by his last pull request. In the meantime, you can work on the first two UI issues.
Mike Klishevich
@klishevich
Aug 07 2016 20:02

I fixed bug "1. If user dismisses popup confirming the change of values on a screen, the values on the screen remain unchanged. " But I have problem passing tests. I got following errors
AccountSettingSection component ✖ should navigate to NewEmail PhantomJS 2.1.1 (Mac OS X 0.0.0) expected 'spy' to be called with arguments { pathname: "/AccountSetting/NewEmail", query: { email: "newmail@mail.com" } } expected false to be true assert@/Users/klisha/np/mxfactorial/test/app/utils/geo.spec.js:13173:38 fail@/Users/klisha/np/mxfactorial/test/app/utils/geo.spec.js:13193:24 value@/Users/klisha/np/mxfactorial/test/app/utils/geo.spec.js:13265:24 /Users/klisha/np/mxfactorial/test/app/components/AccountSetting/AccountSettingSection.spec.js:131:31

routes ✖ should be renderable PhantomJS 2.1.1 (Mac OS X 0.0.0) expected 0 to be 1 assert@/Users/klisha/np/mxfactorial/test/app/utils/geo.spec.js:13173:38 fail@/Users/klisha/np/mxfactorial/test/app/utils/geo.spec.js:13193:24 value@/Users/klisha/np/mxfactorial/test/app/utils/geo.spec.js:13265:24 /Users/klisha/np/mxfactorial/test/app/routes.spec.js:94:116

Max Funk
@mxfactorial
Aug 07 2016 20:06
Create pull request, add commit, comment error, ask @hotay to code review.