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

fix: backport latest single value fixes in master #322

Merged
merged 12 commits into from
Sep 27, 2019

Conversation

janhenrikoverland
Copy link
Collaborator

Backports the latest work on single value charts.

janhenrikoverland and others added 12 commits September 27, 2019 15:12
Layout validation is run when the update
button is clicked.
This should avoid to load the plugin when there is an invalid layout
which will likely cause an error due to missing expected data/config.

Also, in the reducers we cannot always assume the layout has all the
required dimensions/items, ie. when starting a viz from scratch and
switch viz type or click the update button before selecting some
required dimension item.
* fix: single value without dx items

* fix: pass array instead of object

* fix: fix prettier

* fix: code style

* @dhis2/analytics@2.4.0
@janhenrikoverland janhenrikoverland changed the title Fix v33 backport fix: backport latest single value fixes in master Sep 27, 2019
@turban turban self-requested a review September 27, 2019 13:27
Copy link

@turban turban left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dhis2-bot
Copy link
Contributor

🎉 This PR is included in version 33.1.13 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

None yet

4 participants