forked from allegro/turnilo
-
Notifications
You must be signed in to change notification settings - Fork 0
DPBI-523: Merge upstream 1.40.2 #6
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
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Use d3-dsv package for parsing tsv/csv files. We can pass autoType function which converts values from tsv/csv file to javascript types. * Add Unemployment dataset and few e2e smoke tests
* Reorganize Customization as POJsOs. * Move deserializers to client directory, so we don't reference bundle dependencies in server application (allegro#745) * week start configuration (allegro#743) * Locale section in Customization settings * cyclicShift example * Pass Locale to date-range-picker and use to select days names based on locale settings * Export file encoding is specified in Locale configuration. (allegro#747) * Rename /sources endpoint * AppSettings tests * Few more tests
* Add OAuth docs * Typos and stuff
…ro#753) * Return default locale if passed unrecognized locale identifier * Better messaging for locale fallback.
* Show already selected values in StringFilterMenu at the top of the list. These values are also filtered with search text field. Remove unnecessary limit prop in StringValuesList. Parent already limits it's query. * Add `downlevelIteration` to mocha ts-node config so it can still compile code with target es5
* Refactor DataCube * Clear code a bit * Refactor Dimensions * Simplify DataCube types * Remove few TODOs * essene tests * Fix tests * Bunch of tests * Measures refactor
* Dimension has now limits property * Handle limits property in url decoders * Hardcode small limit in resolvers. To be fixed in allegro#756 * Use limits in Split menu * Name expression for brevity * Filter early faulty definitions in view definition converter
…llegro#765) * Custom TimeShift periods and Latest periods in Time Dimension Filter * Fix e2e tests * Test normalizeDurationName and solve few edge cases with regular expressions, sic!
…all Vizualisations (allegro#777)
* Extract data fetching for visualisation into DataProvider component * Move Highlight class * Remove BaseVisualization * Extract ScrolledTable and Interaction controller for Table * Change class selectors in e2e tests * Rename callbacks for brevity * Unused import. File do not use jsx.
* Simpler factories for DragPosition * Better API for checking if we have filter on dimension * PartialTilesProvider for keeping state of partial tiles and providing accessors and setters. Bunch of renames follows. * getIndex helper for DragPosition * Fix typo - turnilo should check filters, not splits when adding new filter. * Drop Placeholder data type and use PartialSeries. * Refactor FilterTiles just like SeriesTiles. * FilterTilesRow is responsible for handling interactions (mostly drag'n' drop) and decorating callbacks from PartialTileProvider * FilterTiles just renders correct tiles. It handles overflow Tile and possible PartialFilter * FilterTile and PartialTilerTile are just visual components. They use WithRef to correctly render Menus below. * FilterMenus do not use clicker directly but accept saveClause callback. That also simplifies some logic in validation - no need to check whole filter, just current clause. Caveat: TimeFilterMenu still uses Clicker to change time shift. Something to improve in the future. * Use new FilterTilesRow * Due to correct calculation of maxItems, all tiles from spec fit into row, so we should not to look for overflow tile. * Bunch of small code review fixes.
…auses and it is impossible to have partial clause. There are only partial items. (allegro#783)
* Fix type: extendedDescription can be undefined * Concatenate description in InfoBubble and pass it from CubeHeaderBar * Add some spacing for paragraphs.
Small refactor for FilterTile. Label is defined as a component for brevity.
* match @types with ts3.5 * done * jsdom * jsdom rollback * js-yaml rollback * safeLoad rollback - pr ready * js-yaml * merge * merge * 3.6 * fix
* Fix audit critical issues * done * merge * post merge * downgrade terser-webpack-plugin * merge * merge * Merge remote-tracking branch 'upstream/master' into audit-critical * audit * done * ready --------- Co-authored-by: Adrian Mróź <[email protected]>
* Add more variables to query logs * add custom logger for query viz * basic tests
* Bump visx to latest v1 * audit fix * merge
Bumps [cypress](https://github.com/cypress-io/cypress) from 12.5.1 to 12.8.1. - [Release notes](https://github.com/cypress-io/cypress/releases) - [Changelog](https://github.com/cypress-io/cypress/blob/develop/CHANGELOG.md) - [Commits](cypress-io/cypress@v12.5.1...v12.8.1) --- updated-dependencies: - dependency-name: cypress dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [cypress](https://github.com/cypress-io/cypress) from 12.8.1 to 12.9.0. - [Release notes](https://github.com/cypress-io/cypress/releases) - [Changelog](https://github.com/cypress-io/cypress/blob/develop/CHANGELOG.md) - [Commits](cypress-io/cypress@v12.8.1...v12.9.0) --- updated-dependencies: - dependency-name: cypress dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [cypress](https://github.com/cypress-io/cypress) from 12.9.0 to 12.10.0. - [Release notes](https://github.com/cypress-io/cypress/releases) - [Changelog](https://github.com/cypress-io/cypress/blob/develop/CHANGELOG.md) - [Commits](cypress-io/cypress@v12.9.0...v12.10.0) --- updated-dependencies: - dependency-name: cypress dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [cypress](https://github.com/cypress-io/cypress) from 12.10.0 to 12.13.0. - [Release notes](https://github.com/cypress-io/cypress/releases) - [Changelog](https://github.com/cypress-io/cypress/blob/develop/CHANGELOG.md) - [Commits](cypress-io/cypress@v12.10.0...v12.13.0) --- updated-dependencies: - dependency-name: cypress dependency-type: direct:development update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Basic implementation * more tests and docs
Release 1.40.2
|
⏱️ Lighthouse report |
tvamsisai
approved these changes
Nov 15, 2023
tvamsisai
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
sean-traynor-98
approved these changes
Nov 23, 2023
sean-traynor-98
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Upgrading Turnilo to latest stable version 1.40.2