rehanvdm/serverless-website-analytics

Add option to view by hour/day

rehanvdm opened this issue · 4 comments

This is currently decided for you by the logic here

If it is < 3 days then it will get data by the hour otherwise by the day.

Sometimes you want to look at a weeks data and see spikes to identify when traffic orginated

Make this a drop-down on the chart

may i attempt this?

@rohitverma007 Yes that would be great thanks. Can you maybe hold off starting on it till next week?

I am busy with #9 and have refactored the chart component, so once that is merged I will ping you.

@rohitverma007 That work is pretty much done #59 but I am still to self-review the PR. You can branch off the add-events branch when you start this feature.

So you will see that there is a selector prop on the chart component already and that I think you could just make that an array and then handle the click and everything else similar to what the existing one does.

image

Let me know if you require assistance/pain points. Not sure if I have documented enough in https://github.com/rehanvdm/serverless-website-analytics/blob/main/docs/CONTRIBUTING.md

I don't think I have. But you can locally spin up the backend and frontend for development.

npm install
cd src/src/ && npm install 
cd src/src/frontend && npm install 

For testing locally, it will require you to use a deployed serverless-website-analytics component. Fill in your data in the tsrc/src/test-config.ts file.

Then to run the backend:

cd src/src
npm run start-local-api-front

To run the frontend:

cd src/src/frontend
npm run dev

Actually, I see the contributing doc had that in 😅. That feature has been done and is merged now. There shouldn't be any big changes like this one again. Let me know if you still want to do this one?