Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

15 add spending pie chart #35

Closed
wants to merge 16 commits into from
Closed

15 add spending pie chart #35

wants to merge 16 commits into from

Conversation

Andze
Copy link

@Andze Andze commented Dec 7, 2016

No description provided.

RaymondCM and others added 16 commits November 20, 2016 13:45
Now shows how many pervious visits and the average spend.
(#1) Add analytic breakdown to overview component
Clear local storage one error to prevent error loop
(#4) Add root redirection when access key exists
Analytics page added to the navigation bar and also linked successfully.
Page is currently blank, ready for content.
(#5) Add average spend to analytics
(#10) Add most and least spent module to Analystics page
(#9) Add most visited to analytics page
@Andze Andze closed this Dec 7, 2016
@robcalcroft
Copy link
Owner

I assume as you closed this PR it's not ready for merging?

@robcalcroft
Copy link
Owner

Tagging #1 in this PR as feature being created sounds like #1

@Andze
Copy link
Author

Andze commented Dec 8, 2016

Yeah sorry about that, we are still adding a few features to the page and cleaning up code. We should be done next week then we will submit the pull request. This is what the page looks like currently

monzo development so far

@zoetrope69
Copy link
Collaborator

you should keep your PR open as you work on it so people can give you feedback

@robcalcroft robcalcroft reopened this Jan 8, 2017
@robcalcroft
Copy link
Owner

@Andze do you still want to merge this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants