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

feat: support options for multi axes (DHIS2-6672) #890

Merged
merged 16 commits into from
Jun 1, 2021

Conversation

martinkrulltott
Copy link
Contributor

@martinkrulltott martinkrulltott commented Apr 13, 2021

Implements DHIS2-6672

Relates to dhis2/data-visualizer-app#1692

Requires dhis2/app-platform#552


Key features

  1. Support options for multi axis

Description

Combines the handling of single axis and multi axis into a uniformed axis handling, to enable all options regardless of how many axes are used.

dhis2/app-platform#552 is needed for the plural handling to work properly.

Generates axis titles based on if multi axes are used and/or the amount of data items are in use, see the screenshots in the DV PR for more info on what it looks like.

@martinkrulltott martinkrulltott marked this pull request as ready for review May 31, 2021 08:59
@martinkrulltott martinkrulltott requested a review from edoardo May 31, 2021 08:59
martinkrulltott and others added 2 commits June 1, 2021 11:52
…index.js

Co-authored-by: Edoardo Sabadelli <edoardo@dhis2.org>
…index.js

Co-authored-by: Edoardo Sabadelli <edoardo@dhis2.org>
@martinkrulltott martinkrulltott merged commit a3ad56e into master Jun 1, 2021
@martinkrulltott martinkrulltott deleted the feat/DHIS2-6672-multi-axis-options branch June 1, 2021 11:59
dhis2-bot added a commit that referenced this pull request Jun 1, 2021
# [18.2.0](v18.1.4...v18.2.0) (2021-06-01)

### Features

* support options for multi axes (DHIS2-6672) ([#890](#890)) ([a3ad56e](a3ad56e))
@dhis2-bot
Copy link
Contributor

🎉 This PR is included in version 18.2.0 🎉

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
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants