Skip to content
This repository has been archived by the owner on Nov 4, 2024. It is now read-only.

Release/2.51.0 #4311

Merged
merged 13 commits into from
Jul 6, 2022
Merged

Release/2.51.0 #4311

merged 13 commits into from
Jul 6, 2022

Conversation

jeroen-drenth
Copy link
Contributor

No description provided.

jeroen-drenth and others added 13 commits June 20, 2022 13:22
* Refactored axes component

* Lint fixes

* Removed number as T extend

* Removed redundant prop

* Code optimalization
* chore: update styled-components and friends

* chore: update typescript for app

* chore: update typescript for cli

* chore: update ts-node, sanity, typescript

* fix: fall back to variantName in json when not provided in sanity

* chore: audit fix

* restore data-structure

Co-authored-by: VWSCoronaDashboard24 <[email protected]>
* fix: remove minHeight for vr and gm pages choropleth

* fix: only set minHeight when provided

* fix: remove console.log

* fix: code cleanup

Co-authored-by: VWSCoronaDashboard24 <[email protected]>
- Align label next to top axis-left tick value
- Align top axis left tick value with chart line
- Some code formatting
* feat: scroll through list of results with keyboard

* feat: Add guard pattern and named functions

- Change if statement and add guard
- Add extra comment for specific calculations
- Change functions to named functions

* feat: rename function according to coding standards
* feat: show sanity debug toggle also ons taging

* fix: revert code change

* fix: revert code change

Co-authored-by: VWSCoronaDashboard24 <[email protected]>
The graph was unarchived but still pointed to the archived dataset. This commit fixes that. And properly removes it from other calculations as well.

Co-authored-by: VWSCoronaDashboard25 <[email protected]>
Prevents a potential runtime error since #4292 by returning empty x-axis labels if the graph has no values.

Co-authored-by: VWSCoronaDashboard25 <[email protected]>
* Removed empty tile

* Removed unused import

* Removed prop and type
@jeroen-drenth jeroen-drenth changed the base branch from develop to master July 6, 2022 13:52
Copy link
Contributor

@VWSCoronaDashboard25 VWSCoronaDashboard25 left a comment

Choose a reason for hiding this comment

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

LGTM 😁

@jeroen-drenth jeroen-drenth merged commit b7b62bb into master Jul 6, 2022
@jeroen-drenth jeroen-drenth deleted the release/2.51.0 branch July 6, 2022 14:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants