-
Notifications
You must be signed in to change notification settings - Fork 38
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
Sync 7.10 branch with reporting repo #33
Merged
joshuali925
merged 307 commits into
opensearch-project:aes-7.10-Aug-patch
from
joshuali925:aes-7.10-Aug-patch
Jan 13, 2023
Merged
Sync 7.10 branch with reporting repo #33
joshuali925
merged 307 commits into
opensearch-project:aes-7.10-Aug-patch
from
joshuali925:aes-7.10-Aug-patch
Jan 13, 2023
Conversation
This file contains 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
* Create to enable pdf/png report download * Add input validation * Add UT
…ypescript Migrate public/main to Typescript
* Bump version number to `7.8.0` * Configure `yarn test:jest` for unit test instead of `yarn test`
Update all user stories in Design Proposal to reflect the new design
…t#34) * Add Key UX flows, screens, supporting detail images * Update ux images + add standard res exports * Update screens and add reporting UX documentation * Create Kibana-Reporting-UX-documentation.md
Migrated create_report/ to Typescript
* APIs to get all reports and single report metadata * stream reports to client
Added UI for Report Details page and snapshot test suite for components.
Refactored landing page UI to reflect final UI. Fixed search functionalities and added filter capabilities as per user story, as well as snapshot test suite for all components.
Migrate front-end to new Kibana platform
Added UI for Report definitions details and made incremental changes to Report details page. Added snapshot test as well
Updated the Final UI for Create Report and added snapshot tests.
* add schema, add delete report by id handler * Add reportDefinition router; CRUD
Add edit report definition UI and updated report definitions file structure
…nsearch-project#51) Hooked in APIs for Reporting homepage and Create report definition. Reporting homepage now displays reports & report definitions from ES, and creating a new report definition will add it to the ES index.
Connected report details to backend to show report details page onClick
Connected report definition details to backend and updated test case to reflect changes
…rch-project#55) * Update version number, changed the elasticsearch client call from legacy to adminClient
Adding API endpoints for data reports
Fix CSV test suite with the pre-requisites needed to pass data report test cases
* change project layout to include reports-scheduler plugin * Add report scheduler to support scheduling feature of kibana reports
…x-dco Add PR template with Linux DCO
This reverts commit 21aebcc.
This reverts commit 7bed87f.
This reverts commit 2c2d724.
Pass processing if doc[field] == null Based on opendistro-for-elasticsearch/kibana-reports#327 (comment)
…ject#366) Signed-off-by: Joshua Li <[email protected]>
…ensearch-project#370) Add missing test case for the fix "pass doc[field] == null"
… Kibana advanced setting (opensearch-project#371)
Signed-off-by: Zhongnan Su <[email protected]>
* Fix csv parsing function, switch json to csv library Signed-off-by: Joshua Li <[email protected]> * Revert "Fix csv parsing function, switch json to csv library" This reverts commit 6b3d88fccd588f7dcd2069e59ecc5bcc1a024a1f. * Fix csv parsing function Signed-off-by: Joshua Li <[email protected]> * Flatten nested results for csv * Add test case for nested fields Signed-off-by: Joshua Li <[email protected]> * Use test data from sample_data_ecommerce * Format code
Signed-off-by: Zhongnan Su <[email protected]>
* Fix ci (#2) Signed-off-by: Joshua Li <[email protected]> * Markdown patch fix (#1) Signed-off-by: David Cui <[email protected]> * Detect iframe, embed, object tags Signed-off-by: Joshua Li <[email protected]> * Disallow redirection to non-localhost urls Signed-off-by: Joshua Li <[email protected]> * Disallow connection to non-allowlisted urls Signed-off-by: Joshua Li <[email protected]> * Disable JIT Signed-off-by: Joshua Li <[email protected]> * Fix localstorage logic Signed-off-by: Joshua Li <[email protected]> * Try to fix CI Signed-off-by: Joshua Li <[email protected]> Signed-off-by: Joshua Li <[email protected]> Signed-off-by: David Cui <[email protected]> Co-authored-by: David Cui <[email protected]>
Signed-off-by: Joshua Li <[email protected]>
* Update timeout & resize the page Signed-off-by: Rupal Mahajan <[email protected]> * Change timeout Signed-off-by: Rupal Mahajan <[email protected]> Signed-off-by: Rupal Mahajan <[email protected]>
* [BACKPORT] Use front-end report generation instead of chromium Signed-off-by: Shenoy Pratik <[email protected]> * fix workflow build Signed-off-by: Shenoy Pratik <[email protected]> * delete puppeteer step from workflow Signed-off-by: Shenoy Pratik <[email protected]> * delete unused kbnn bootstrap step Signed-off-by: Shenoy Pratik <[email protected]> * added yargs 16.2.0 resolution Signed-off-by: Shenoy Pratik <[email protected]> * adding svg-path runtime dependency Signed-off-by: Shenoy Pratik <[email protected]> * fix url for html2canvas Signed-off-by: Shenoy Pratik <[email protected]> * fix relative path for aes url Signed-off-by: Shenoy Pratik <[email protected]> * replace header name Signed-off-by: Shenoy Pratik <[email protected]> Signed-off-by: Shenoy Pratik <[email protected]>
Signed-off-by: Joshua Li <[email protected]>
rupal-bq
approved these changes
Jan 13, 2023
Signed-off-by: Joshua Li <[email protected]>
mengweieric
approved these changes
Jan 13, 2023
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.
Description
Please merge instead of squash this PR
Issues Resolved
[List any issues this PR will resolve]
Check List
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.