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
Testing implementation of wagtailcharts #7250
base: main
Are you sure you want to change the base?
Conversation
You'll need to run |
Run `yarn build` to rebuild the frontend.
b111e2f
to
12e0c23
Compare
@csebianlander I think I got this working thanks to some helpful JS tips from @anselmbradford. Ans, if you have a minute, can you take a glance at my JS and see if it seems reasonable? I'm able to get working charts on a BrowsePage (for example http://localhost:8000/admin/pages/12070/edit/) with a configuration like this: This generates a chart like this: You'll need to |
I've made a lot of notes and have a lot of thoughts about this. The most prevalent is that it's a wonderful improvement from a back-end usability perspective. We can add our CFPB-specific palette using the
Some things I wonder if we can integrate simply into this test:
|
This is a draft PR designed to test the wagtailcharts package as a potential replacement or supplement to our Simple Charts module.
Right now, the rendering template is not working, because of reasons presumably beyond my understanding or expertise, but you can add, edit, and save the block properly in the page editor (I've only added it to the Browse page type for now just for testing purposes).