Working on analytics data for post's traffic #132
Replies: 3 comments
-
The ui looks great overall but using more red in the charts would better match the theme. Additionally, if we use tailwind colors for the charts that require colors (assuming you havent used them yet), it would be more consistent since the entire ui is built with Tailwind. I think the classic view count should remain, and detailed analytics should only be accessible for post authors, while normal users should only see the view count. Other than that, everything looks fantastic. Great work, and thanks so much for this awesome contribution to repo. If you dont want to spend time tweaking the ui, I would be happy to accept your contribution and make the necessary improvements by myself. |
Beta Was this translation helpful? Give feedback.
-
thanks for your complements🙂, I haven't used tailwind in charts yet, i'm leaving ui up to you, soon i will create PR, i want ask question, i'm using context processor function for fetching line graph data in backend does it good practice or should i create an endpoint for this? |
Beta Was this translation helpful? Give feedback.
-
Im looking forward to your PR. I will make the necessary adjustments for the UI. About your question, context processors work well for small template data, but an API endpoint might be a better approach for graph data. But let me see the current code, maybe I was wrong. |
Beta Was this translation helpful? Give feedback.
-
i want suggestions regarding ui and existing code changes
if ui doesn't look good then give me another ui sample
and should i need to remove existing page view count logic because it can be done via new code which work for analytics data
flaskBlog.-.Personal.-.Microsoft_.Edge.2025-03-05.21-41-32.webm
if you have any more tip about this please tell me
Beta Was this translation helpful? Give feedback.
All reactions