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

Charts #242

Open
wants to merge 32 commits into
base: master
Choose a base branch
from
Open

Charts #242

wants to merge 32 commits into from

Conversation

illa
Copy link

@illa illa commented Sep 18, 2016

What do you think about adding charts to reports?

I forked your repo and:

  1. added tab 'chart options' where user can choose type of chart (columns, line etc) and what data should be on x and y axis
  2. added tab 'chart' (in tab 'report') where user can see and download chart as png file
  3. changed report model to save chart options

Here are few screenshots presenting new feature.

Simple chart:
example00
example01
example02

Chart with few series and complex categories:
example10
example11
example12

Chart with series names from values in rows (not column names):
example20
example21
example22

illa added 30 commits September 5, 2016 20:55
categories: first colum
series: second colum
values on y: last column.
Chart Options tab is not working properly now.
It works only locally now - chart information in database doesn't change yet.
(When chart of type 3 is selected).
@MiltonLn
Copy link

MiltonLn commented Oct 4, 2016

This sounds like an awesome feature!

@predatell
Copy link
Contributor

Is it difficult to add this feature as separated package with requirement of django-report-builder?

@bufke
Copy link
Member

bufke commented Jan 26, 2017

I'd only be willing to merge this if someone can step up and help maintain this project. Otherwise I'd like to freeze all features and focus on stability and running in newer versions of Django.

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

Successfully merging this pull request may close these issues.

4 participants