goldbergyoni / node-chaos-monkey

Extremly naughty chaos monkey for Node.js
MIT License
228 stars 10 forks source link

API usage is ready #3

Open goldbergyoni opened 5 years ago

goldbergyoni commented 5 years ago

@TITAN9389 I pushed a code that serves all API routes with mock data and updated the documentation readme.md

On the repo homepage, please read only the part that starts with: "API usage".

Basically, you can just perform 'npm start' and approach the routes.

Is the documentation detailed enough?

cc @amaliaman

sebmaz93 commented 5 years ago

@i0natan Yes its all good thanks.

sebmaz93 commented 5 years ago

@i0natan there is the CORS error , not allowing my localhost:3000 , i don't want to change your node code if you can do it later i'm using Chrome extension now to pass it.

goldbergyoni commented 5 years ago

@TITAN9389 OK, adding now

goldbergyoni commented 5 years ago

@TITAN9389 I added not only CORS but also serving static files from the "UI" folder.

So you can work on a different port, but also try to locate your files within the UI folder, build for production, and try serving from the server port. This would be the real world scenario so I suggest testing it. Kindly pull latest and LMK wether works for you?

Any estimate when you might be able to share a draft?

sebmaz93 commented 5 years ago

yes okay Thanks ,i'll see and do the Build files as well , I'll share this later today @i0natan

sebmaz93 commented 5 years ago

@i0natan i've connected to web socket ,but i'm getting Null data sometimes is that normal or what ? image

goldbergyoni commented 5 years ago

@TITAN9389 Not normal, probably server side issue but I guess it doesn't block you at all or does it?

sebmaz93 commented 5 years ago

@i0natan not blocking me no ,but can make problems if the data is NULL i can't render stuff

sebmaz93 commented 5 years ago

@i0natan as i knew the NULL throwing error while rendering the UI , so i catch it now with error so just UI will ignore and just render the data. and the data keeps emitting is it like that in normal ? i mean if i let it open it will still send me data forever 9999...we can have a call tomorrow to explain everything better. image

goldbergyoni commented 5 years ago

@TITAN9389 Currently there's no real data, it's a mock that throw infinite events, that won't be that way in real-scenario - there won't be more than 10 items. Just ignore it.

Any chance you can share some prtscn, video clip or push code? just want to get a sense of how the UI looks like now

sebmaz93 commented 5 years ago

i pushed the code to my Branch , merge it and run npm start from ui folder ( didn't make build yet ).

sebmaz93 commented 5 years ago

@i0natan @amaliaman made some changes in the code (webapp.js) file now just run npm run build from UI folder then just npm start from main folder it will serve the build files.