Sync Feature/point in time branch with main #3273
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
alchemist51
force-pushed
the
feature/point-in-time
branch
from
bf1628a to
313d062
Compare
Codecov Report
📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more @@ Coverage Diff @@
## feature/point-in-time #3273 +/- ##
=========================================================
- Coverage 66.67% 66.46% -0.22%
=========================================================
Files 3220 3206 -14
Lines 61452 61565 +113
Branches 9417 9497 +80
=========================================================
- Hits 40975 40917 -58
- Misses 18231 18377 +146
- Partials 2246 2271 +25
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
alchemist51
force-pushed
the
feature/point-in-time
branch
from
313d062 to
4104658
Compare
|
@Arpit-Bandejiya can you update the PR description to include what are the changes being introduced so that its easier for the reviewers to review the change? If you could also include screenshots and videos of the change, that will immensely help the PR process. |
I think its goal is to just update their feature branch with the latest from If this the case @Arpit-Bandejiya I can assist you with this. |
|
@Arpit-Bandejiya Should this PR be closed? |
|
Hi this is for merging the latest into the branch @kavilla can you please assist with the failing tests? Thanks! |
alchemist51
force-pushed
the
feature/point-in-time
branch
2 times, most recently
from
2e3f5f1 to
dabc10f
Compare
…pensearch-project#2964) * When updating the timestamps were requested, scanCopy first set the times and then copied content into the destination folder. On certain platforms, copying files into a folder updates its "Last Access Time" and that overwrites the just set timestamps. This PR, makes sure the timestamps are set only after copying the content. Signed-off-by: Miki <[email protected]> Signed-off-by: Miki <[email protected]> Signed-off-by: Arpit Bandejiya <[email protected]>
…ch-project#2965) Signed-off-by: Miki <[email protected]> Signed-off-by: Miki <[email protected]> Signed-off-by: Arpit Bandejiya <[email protected]>
…roject#2963) * Makes `yarn pkg-version` work across platforms Signed-off-by: Miki <[email protected]> * Makes `yarn pkg-version` work across platforms Signed-off-by: Miki <[email protected]> Signed-off-by: Miki <[email protected]> Signed-off-by: Arpit Bandejiya <[email protected]>
Signed-off-by: Miki <[email protected]> Signed-off-by: Miki <[email protected]> Signed-off-by: Arpit Bandejiya <[email protected]>
…ross platforms (opensearch-project#2994) Signed-off-by: Miki <[email protected]> Signed-off-by: Miki <[email protected]> Signed-off-by: Arpit Bandejiya <[email protected]>
Bumps [loader-utils](https://github.com/webpack/loader-utils) from 2.0.3 to 2.0.4. - [Release notes](https://github.com/webpack/loader-utils/releases) - [Changelog](https://github.com/webpack/loader-utils/blob/v2.0.4/CHANGELOG.md) - [Commits](webpack/loader-utils@v2.0.3...v2.0.4) --- updated-dependencies: - dependency-name: loader-utils dependency-type: direct:development ... Signed-off-by: dependabot[bot] <[email protected]> Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Signed-off-by: Arpit Bandejiya <[email protected]>
Fixes opensearch-project#2996, opensearch-project#2353 - Bump `leaflet-vega` to `^0.9.0` - they merged in our upstream PR and made a release - Fix the parameter name for passing `vega.parse` options. Signed-off-by: Miki <[email protected]> Signed-off-by: Arpit Bandejiya <[email protected]>
Basic data points [OSD] 16 submitted PRs (https://github.com/opensearch-project/OpenSearch-Dashboards/pulls/manasvinibs) [OSD] 74 reviewed PRs (https://github.com/opensearch-project/OpenSearch-Dashboards/issues?q=reviewed-by%3Amanasvinibs) [OSD] 28 issues involved (https://github.com/opensearch-project/OpenSearch-Dashboards/issues?page=1&q=involves%3Amanasvinibs+is%3Aissue) Highlight Mana is assisting with extensions project which will be the next evolution of extending core functionality from OpenSearch Dashboards Mana implemented opensearch-project#2734 which allows for a huge quality of life for local development for external plugin developers to utilize snapshots with a single CLI command compared to before when they would had to pull down OpenSearch build, install their plugin on OpenSearch, and ensure the proper configurations. This has caused historically problems when plugin teams do development and miss some steps per their onboard documentation/PR suggestion and get different results than expected. Mana has assisted reviewing PRs providing great insight on BWC tests, BWC in general, and the release process. Mana has added documentation from insight she has gained within the informal dev doc repo https://cptnb.github.io/opensearch-dashboards-dev-docs/ ensuring the spread of knowledge. Signed-off-by: Kawika Avilla <[email protected]> Signed-off-by: Kawika Avilla <[email protected]> Signed-off-by: Arpit Bandejiya <[email protected]>
…-project#2896) * Add global data persistence for vis builder Persist filters, time range, time refresh interval for vis builder when we refresh or navigate to other apps such as dashboard, discover, timeline and visualize Signed-off-by: abbyhu2000 <[email protected]> Signed-off-by: Arpit Bandejiya <[email protected]>
…t#2990) Signed-off-by: Miki <[email protected]> Signed-off-by: Arpit Bandejiya <[email protected]>
…true (opensearch-project#3020) `no-restricted-paths` compares source files and import statements, and their membership in restricted zones. However, when `allowSameFolder` is true, it failed to remove a trailing slash before validation which results in a false-positive. Signed-off-by: Miki <[email protected]> Signed-off-by: Miki <[email protected]> Signed-off-by: Arpit Bandejiya <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>
* test connection intial code Signed-off-by: mpabba3003 <[email protected]> * error handling Signed-off-by: mpabba3003 <[email protected]> * refactor Signed-off-by: mpabba3003 <[email protected]> * removing get cluster info dependency Signed-off-by: mpabba3003 <[email protected]> * refactor test connection Signed-off-by: mpabba3003 <[email protected]> * adding test cases and test connection on edit datasource Signed-off-by: mpabba3003 <[email protected]> * adding changelog comment Signed-off-by: mpabba3003 <[email protected]> * fixing bug on edit datasource page Signed-off-by: mpabba3003 <[email protected]> * refactor based on PR comments Signed-off-by: mpabba3003 <[email protected]> Signed-off-by: mpabba3003 <[email protected]> Signed-off-by: Arpit Bandejiya <[email protected]>
Signed-off-by: Josh Romero <[email protected]> Signed-off-by: Arpit Bandejiya <[email protected]>
* [Chore] Add 2.4.1 release notes Signed-off-by: Josh Romero <[email protected]> Signed-off-by: Arpit Bandejiya <[email protected]>
…t#2918) Currently, the new table can not format Url. If we set to use URL format in index pattern field, table will display it as string. In this PR, we switch the format from string to html. To make html understandable by react as a DOM element, we use dangerouslySetInnerHTML to transform it. For the security, since the content is not from random input but fetched from stored data, we should be safe as long as data is not messed. To provide more security protection, we also add dompurify package to sanitize the html content. Issue Resolved: opensearch-project#2905 Signed-off-by: Anan Zhuang <[email protected]> Signed-off-by: Anan Zhuang <[email protected]> Signed-off-by: Arpit Bandejiya <[email protected]>
* Removes manual resolution of `axios`. Signed-off-by: Tommy Markley <[email protected]> Signed-off-by: Tommy Markley <[email protected]> Signed-off-by: Arpit Bandejiya <[email protected]>
* The minimatch resolution was no longer necessary after the upstream library that depended on v3.0.4 was removed in opensearch-project#2711. Signed-off-by: Tommy Markley <[email protected]> Signed-off-by: Tommy Markley <[email protected]> Signed-off-by: Arpit Bandejiya <[email protected]>
Bump hapi/statehood to 7.0.4 to solve security concerns. Issue Resolved: opensearch-project#3406 Signed-off-by: Anan Zhuang <[email protected]> Signed-off-by: Arpit Bandejiya <[email protected]>
…oject#3409) Signed-off-by: Su <[email protected]> Co-authored-by: Anan Zhuang <[email protected]> Signed-off-by: Arpit Bandejiya <[email protected]>
* [CVE-2023-25166] Bumps formula to 3.0.1 Signed-off-by: Su <[email protected]> Co-authored-by: Miki <[email protected]> Co-authored-by: Anan Zhuang <[email protected]> Signed-off-by: Arpit Bandejiya <[email protected]>
Introduce new ui setting for custom vector map's size parameter. The default value is 1000. Users can increase this limit by updating this value in Advanced Settings. Signed-off-by: Vijayan Balasubramanian <[email protected]> Signed-off-by: Arpit Bandejiya <[email protected]>
…n't be GC-ed (opensearch-project#3390) The search telemetry was disabled by default, there is a issue when search telemetry read configuration and creates an Observable object that won't be GC-ed. Signed-off-by: Tao Liu <[email protected]> Signed-off-by: Arpit Bandejiya <[email protected]>
* Corrects NOTICE file copyright date range * Follow-up on opensearch-project#3051 Resolves opensearch-project#765 Signed-off-by: Tommy Markley <[email protected]> * Update CHANGELOG.md Co-authored-by: Miki <[email protected]> Signed-off-by: Tommy Markley <[email protected]> Co-authored-by: Zhongnan Su <[email protected]> Co-authored-by: Miki <[email protected]> Signed-off-by: Arpit Bandejiya <[email protected]>
…project#3358 (opensearch-project#3359) * Upgrade vega-tooltip to 0.30.0 to support custom tooltips opensearch-project#3358 Signed-off-by: Jovan Cvetkovic <[email protected]> * Upgrade vega-tooltip to 0.30.0 to support custom tooltips opensearch-project#3358 Signed-off-by: Jovan Cvetkovic <[email protected]> * Upgrade vega-tooltip to 0.30.0 to support custom tooltips opensearch-project#3358 Signed-off-by: Jovan Cvetkovic <[email protected]> --------- Signed-off-by: Jovan Cvetkovic <[email protected]> Signed-off-by: Arpit Bandejiya <[email protected]>
* Created untriaged issue workflow. Signed-off-by: dblock <[email protected]> * Remove existing untriaged workflow with wrong name Signed-off-by: Josh Romero <[email protected]> --------- Signed-off-by: dblock <[email protected]> Signed-off-by: Josh Romero <[email protected]> Co-authored-by: Josh Romero <[email protected]> Signed-off-by: Arpit Bandejiya <[email protected]>
…ct#3058) * [Multiple DataSource] Add support for SigV4 authentication Signed-off-by: Su <[email protected]> Signed-off-by: Arpit Bandejiya <[email protected]>
…oject#3402) With this change, the `engines.node` of `package.json` need not be a fixed version; it can be a range using a subset of formats defined by `semver`. A very limited subset of what [semver](https://github.com/npm/node-semver/blob/cb1ca1d5480a6c07c12ac31ba5f2071ed530c4ed/README.md#ranges) supports has been added; only a single comparator, composed of an operator and a version, is supported. The supported operators are * `>` Greater than * `>=` Greater than or equal to * `=` Equal * `~` Tilde ranges: Allows patch changes if a minor version is specified but if only a major version is specified, it allows minor changes. * `^` Caret ranges: Allows patch and minor updates when major is non-zero (and we will never have that). If no operator is specified, equality is assumed. Signed-off-by: Miki <[email protected]> Signed-off-by: Arpit Bandejiya <[email protected]>
Signed-off-by: Arpit Bandejiya <[email protected]>
…rch (opensearch-project#3443) * upgrade yarn version to 1.22.10 to be compatible with @openearch-project/opensearch Signed-off-by: Derek Ho <[email protected]> Signed-off-by: Arpit Bandejiya <[email protected]>
…tisfies `engines.node` (opensearch-project#3467) * While building distributables, Node.js runtime is downloaded to be placed in the archivea. This logicwas modified to honor a range for `engines.node` by fetching the latest release of Node.js that satisfied the range. * Some tests covering the build, read a version from `.node-version` to compare with the results of actual function runs; these were changed to either use mocked values or honor the range and use the latest Node.js version. * Some variable and functions referred to `engines.node` as a version; they were corrected to call it a range. Signed-off-by: Miki <[email protected]> Signed-off-by: Arpit Bandejiya <[email protected]>
…pensearch-project#3463) * `engines.node` is relaxed from `14.20.1` to `^14.20.1`: yarn and OSD will allow versions 14.20.1 or greater, but less than 15, to be used but do not impose upgrading to avoid being a breaking change. Users will be able to install any version of Node.js that satisfies `^14.20.1`, moving ahead without waiting for a change in OSD when new versions are released. * `.nvmrc` is bumped to the latest security patch: `14.21.3` Signed-off-by: Miki <[email protected]> Signed-off-by: Arpit Bandejiya <[email protected]>
…-project#3456) Signed-off-by: Su <[email protected]> Signed-off-by: Arpit Bandejiya <[email protected]>
* a Signed-off-by: Anan Zhuang <[email protected]> * Add 2.6 release note Signed-off-by: Anan Zhuang <[email protected]> * Update release-notes/opensearch-dashboards.release-notes-2.6.0.md Co-authored-by: Josh Romero <[email protected]> --------- Signed-off-by: Anan Zhuang <[email protected]> Co-authored-by: Josh Romero <[email protected]> Signed-off-by: Arpit Bandejiya <[email protected]>
…ize (opensearch-project#3421) * docs(DEVELOPER_GUIDE): Make getting started comprehensive and reorganize Enhance getting started guide for new contributors - Add key technology breakdown with links to code guidelines - Add robust prerequisite guide - Clarify forking workflow - Explain bootstrapping - Distinguish between recommended and optional/alternative steps - Add OS-specific guidance - Add tarball usage instructions Make other developer guide improvements: - Update TOC - Remove callout linking to EUI docs - Reorder code guidelines - Standardize header capitalization and make usage guide headers imperative fixes opensearch-project#3396 Signed-off-by: Josh Romero <[email protected]> Signed-off-by: Josh Romero <[email protected]> Co-authored-by: Miki <[email protected]> Signed-off-by: Arpit Bandejiya <[email protected]>
Doc link was invalid and causing an undefined error. This prevented the UI from mounting properly. Issue(s) resolved: * opensearch-project#3498 * opensearch-project#3327 Signed-off-by: Kawika Avilla <[email protected]> Signed-off-by: Arpit Bandejiya <[email protected]>
Signed-off-by: Miki <[email protected]> Signed-off-by: Arpit Bandejiya <[email protected]>
Fixes a small typo in TSVB README.md file. Issue: n/a Signed-off-by: Kawika Avilla <[email protected]> Signed-off-by: Arpit Bandejiya <[email protected]>
Ran npx browserslist@latest --update-db to update caniuse package so the integration tests will pass. Issue: n/a Will be fixed in: opensearch-project#2329 Signed-off-by: Kawika Avilla <[email protected]> Signed-off-by: Arpit Bandejiya <[email protected]>
…nsearch-project#3533) Bump vega from 5.22.1 to 5.23.0. This will also bump vega-function from 5.13.0 to 5.13.1. Issue Resolved: opensearch-project#3526 opensearch-project#3525 Signed-off-by: Anan Zhuang <[email protected]> Signed-off-by: Arpit Bandejiya <[email protected]>
Enable the downloading of Darwin for running the command `yarn opensearch snapshot`. Darwin is not officially supported but snapshots are being built here: https://build.ci.opensearch.org/job/distribution-build-opensearch/ Issue resolved: opensearch-project#2944 Signed-off-by: Kawika Avilla <[email protected]> Co-authored-by: Josh Romero <[email protected]> Signed-off-by: Arpit Bandejiya <[email protected]>
* Follow-up from opensearch-project#3018 Signed-off-by: Tommy Markley <[email protected]> Signed-off-by: Arpit Bandejiya <[email protected]>
) * point in time management plugin and empty state Signed-off-by: Ajay Gupta <[email protected]> * updated to new license header Signed-off-by: Ajay Gupta <[email protected]> * [Vis Builder] Rename wizard on save modal and visualization table (opensearch-project#2645) * Rename wizard on save modal and visualization table Change the wizard reference in save modal title, toggle and visualization table Signed-off-by: abbyhu2000 <[email protected]> * Save visualization on save modal, VisBuilder for chart type Signed-off-by: abbyhu2000 <[email protected]> * No experimental flag Signed-off-by: abbyhu2000 <[email protected]> Signed-off-by: abbyhu2000 <[email protected]> Signed-off-by: Ajay Gupta <[email protected]> * [@osd/cross-platform] Adds cross-platform helpers (opensearch-project#2681) Signed-off-by: Miki <[email protected]> Signed-off-by: Miki <[email protected]> Signed-off-by: Ajay Gupta <[email protected]> * Change save object type, wizard id and name to visBuilder (opensearch-project#2673) * Change save object type, wizard ID and Name to visBuilder Signed-off-by: abbyhu2000 <[email protected]> * Rename wizard in functional tests Signed-off-by: abbyhu2000 <[email protected]> * Finalize plugin-id and plugin-name and saved-object-type Signed-off-by: abbyhu2000 <[email protected]> * Change official name to VisBuilder Signed-off-by: abbyhu2000 <[email protected]> * Delete migration Signed-off-by: abbyhu2000 <[email protected]> * Fix functional test Signed-off-by: abbyhu2000 <[email protected]> * Update snapshot Signed-off-by: abbyhu2000 <[email protected]> Signed-off-by: abbyhu2000 <[email protected]> Signed-off-by: Ajay Gupta <[email protected]> * [CVE] Bump loader-utils to 2.0.3 to fix CVE-2022-37601 (opensearch-project#2689) * [CVE] Bump loader-utils to 2.0.3 to fix CVE-2022-37601 Signed-off-by: Zilong Xia <[email protected]> * Update CHANGELOG.md Signed-off-by: Zilong Xia <[email protected]> Signed-off-by: Ajay Gupta <[email protected]> * Add extension point in saved object management to register namespaces and show filter (opensearch-project#2656) * Create filter registry for saved object management to make filters extensible Signed-off-by: Craig Perkins <[email protected]> * WIP on making fetchCounts generic Signed-off-by: Craig Perkins <[email protected]> * First step at making scroll_counts generic Signed-off-by: Craig Perkins <[email protected]> * Work on getting other filter counts with same object count endpoint Signed-off-by: Craig Perkins <[email protected]> * Get tenant count options to display Signed-off-by: Craig Perkins <[email protected]> * Extend find to work with namespaces for saved objects Signed-off-by: Craig Perkins <[email protected]> * Add missing filterFields Signed-off-by: Craig Perkins <[email protected]> * Update jest tests Signed-off-by: Craig Perkins <[email protected]> * Update saved_objects_table snapshot Signed-off-by: Craig Perkins <[email protected]> * Append index to id to make unique Signed-off-by: Craig Perkins <[email protected]> * Add semi-colon Signed-off-by: Craig Perkins <[email protected]> * Fix saved objects table tests with new id scheme Signed-off-by: Craig Perkins <[email protected]> * Only append idx on config type to ensure Advanced Settings have a unique id across tenants Signed-off-by: Craig Perkins <[email protected]> * Remove itemsClone in favor of showing only Advanced Settings of current tenant Signed-off-by: Craig Perkins <[email protected]> * Revert snapshots in table.test.tsx Signed-off-by: Craig Perkins <[email protected]> * Add additional parse_query test Signed-off-by: Craig Perkins <[email protected]> * Add comma Signed-off-by: Craig Perkins <[email protected]> * Create namespaceRegistry to decouple security dashboards plugin and osd core Signed-off-by: Craig Perkins <[email protected]> * Add ability to register an alias Signed-off-by: Craig Perkins <[email protected]> * Update parse query and add to CHANGELOG Signed-off-by: Craig Perkins <[email protected]> * Remove commented out code Signed-off-by: Craig Perkins <[email protected]> * Address code review comments Signed-off-by: Craig Perkins <[email protected]> * Override i18n if alias is regitered Signed-off-by: Craig Perkins <[email protected]> Signed-off-by: Craig Perkins <[email protected]> Signed-off-by: Ajay Gupta <[email protected]> * Removed Leftover X Pack references opensearch-project#2517 (opensearch-project#2638) x-pack references in the code are removed as per the given files in opensearch-project#2517 Issue Resolved: opensearch-project#2517 Signed-off-by: vimal K <[email protected]> Signed-off-by: Ajay Gupta <[email protected]> * Revert "[@osd/cross-platform] Adds cross-platform helpers (opensearch-project#2681)" (opensearch-project#2694) This reverts commit 887093d. Now that downstream plugins and projects are unblocked from builds, we want to make these changes following our standard processes and automated checks Signed-off-by: Josh Romero <[email protected]> Signed-off-by: Josh Romero <[email protected]> Signed-off-by: Ajay Gupta <[email protected]> * [MD] Add design documents of multiple data source feature (opensearch-project#2538) * Add MD design documents, including high level design, user stories, client management detailed design Signed-off-by: Su <[email protected]> Signed-off-by: Ajay Gupta <[email protected]> * Adds @osd/cross-platform (opensearch-project#2703) * Adds helper functions, @osd/cross-platform, to work around the differences of platforms Signed-off-by: Miki <[email protected]> Signed-off-by: Ajay Gupta <[email protected]> * [MD]Update MD data source documentation link (opensearch-project#2693) Signed-off-by: Kristen Tian <[email protected]> Signed-off-by: Kristen Tian <[email protected]> Signed-off-by: Ajay Gupta <[email protected]> * Add `Skip-Changelog` label to skip changelog verification on certain PRs (opensearch-project#2726) Signed-off-by: Miki <[email protected]> Signed-off-by: Miki <[email protected]> Signed-off-by: Ajay Gupta <[email protected]> * Enable visbuilder by default (opensearch-project#2725) * Enable visbuilder by default Signed-off-by: Ashwin P Chandran <[email protected]> * Adds changelog entry Signed-off-by: Ashwin P Chandran <[email protected]> Signed-off-by: Ashwin P Chandran <[email protected]> Signed-off-by: Ajay Gupta <[email protected]> * [Save Object Aggregation View] Fix for export all after scroll count response changed in PR#2656 (opensearch-project#2696) * Fix for filterSavedObjectCounts for namespaceRegistry Signed-off-by: Craig Perkins <[email protected]> * Fix saved_objects_table.test.tsx Signed-off-by: Craig Perkins <[email protected]> * Add to CHANGELOG Signed-off-by: Craig Perkins <[email protected]> * Correct reference to namespacesToInclude Signed-off-by: Craig Perkins <[email protected]> * Use filteredTypeCounts Signed-off-by: Craig Perkins <[email protected]> * Use namespaces similar to types for fetchObjects Signed-off-by: Craig Perkins <[email protected]> * Use _all to represent query for all namespaces Signed-off-by: Craig Perkins <[email protected]> * Pass all registered namespaces Signed-off-by: Craig Perkins <[email protected]> * Switch back signature of scroll_count Signed-off-by: Craig Perkins <[email protected]> * Change countOptions to options Signed-off-by: Craig Perkins <[email protected]> * Use not not instead of in Signed-off-by: Craig Perkins <[email protected]> * Filter namespaces to only include namespace that have been registered Signed-off-by: Craig Perkins <[email protected]> * Add filterQuery with tests Signed-off-by: Craig Perkins <[email protected]> * Update license headers and address review comments Signed-off-by: Craig Perkins <[email protected]> Signed-off-by: Craig Perkins <[email protected]> Signed-off-by: Ajay Gupta <[email protected]> * Update backport workflow to ignore changelog conflicts (opensearch-project#2729) Signed-off-by: Miki <[email protected]> Signed-off-by: Miki <[email protected]> Signed-off-by: Ajay Gupta <[email protected]> * Tweak multiple data source design doc (opensearch-project#2724) Signed-off-by: Kristen Tian <[email protected]> Signed-off-by: Kristen Tian <[email protected]> Signed-off-by: Ajay Gupta <[email protected]> * [Chore] Refactor and improve Discover field summaries (opensearch-project#2391) * [Chore] Refactor and improve field summaries * Convert to typescript * Fix types * Add tests Signed-off-by: Josh Romero <[email protected]> * [Test] Update functional test Groups are now naturally sorted by key, which requires selecting a different date filter Signed-off-by: Josh Romero <[email protected]> * [Chore] Add changelog entry Signed-off-by: Josh Romero <[email protected]> * [Chore] Refactor columns passing, fix bugs * pass columns explicitly as props * fix branding in core mocks * fix `toBeUndefined()` usage in tests * remove leftover comment * fix test subject * condense types Signed-off-by: Josh Romero <[email protected]> Signed-off-by: Josh Romero <[email protected]> Signed-off-by: Ajay Gupta <[email protected]> * Removes Add Integration button (opensearch-project#2723) * Removes Add Integration button Issue Resolved: opensearch-project#2684 Signed-off-by: Bandini Bhopi <[email protected]> * Adds changelog entry Signed-off-by: Bandini Bhopi <[email protected]> Signed-off-by: Bandini Bhopi <[email protected]> Signed-off-by: Ajay Gupta <[email protected]> * Prevent backport workflow from running on umerged PRs (opensearch-project#2746) Signed-off-by: Miki <[email protected]> Signed-off-by: Miki <[email protected]> Signed-off-by: Ajay Gupta <[email protected]> * [Vis Builder] Enable VisBuilder cypress tests (opensearch-project#2728) * Enable VisBuilder cypress tests Signed-off-by: Ashwin P Chandran <[email protected]> * Adds changelog and fix env Signed-off-by: Ashwin P Chandran <[email protected]> * Run all tests Signed-off-by: Ashwin P Chandran <[email protected]> Signed-off-by: Ashwin P Chandran <[email protected]> Signed-off-by: Ajay Gupta <[email protected]> * [CI] update backport custom branch name (opensearch-project#2766) GitHub action for backporting got updated to change custom branch to head template. This was causing issues in clean up and skipping steps. Issue: n/a Signed-off-by: Kawika Avilla <[email protected]> Signed-off-by: Ajay Gupta <[email protected]> * [Legacy Maps Plugin] Prevent reverse-tabnabbing (opensearch-project#2540) Signed-off-by: Miki <[email protected]> Signed-off-by: Miki <[email protected]> Signed-off-by: Ajay Gupta <[email protected]> * change geckodriver version to make consistency (opensearch-project#2772) * change geckodriver version to make consistency Issue Resolved: opensearch-project#2771 Signed-off-by: Anan Zhuang <[email protected]> * combine https-proxy-agent in yarn.lock Signed-off-by: Anan Zhuang <[email protected]> Signed-off-by: Anan Zhuang <[email protected]> Signed-off-by: Ajay Gupta <[email protected]> * [Vis Builder] Add an experimental table visualization in vis builder (opensearch-project#2705) * [Vis Builder] Add an experimental table visualization in vis builder In this PR, we hook up an experimental table vis in vis builder. This table vis is a refactor of previous table. It is written in React and DataGrid component. In this PR, we did two main things: * add an experimental table visualization * enable it in vis builder Issue Resolved (hook up table in vis builder): opensearch-project#2704 The experimental table vis has all the features from current table, including * restore table vis in react using a Datagrid component * datagrid component does not support splitted grids. For future transfer to OUI Datagrid, we create a tableGroup in visData for splitted grids. * restore basic pagenation, sort and format. * implement datagrid columns * display column title correctly * deangular and re-use formatted column * convert formatted column to data grid column * restore filter in and filter out value functions * format table cell to show Date and percent * restore showTotal feature: it allows table vis to show total, avg, min, max and count statics on count * restore export csv feature to table vis * split table in rows and columns Beside of restoring original features, there are some changes: * [IMPROVE] remove repeated column from split tables Currently, when we split table by columns, the split column is shown both in the table title and as a separate column. This is not needed. In this PR, we remove the repeated column in split tables in col. * [NEW FEATURE] adjustable table column width In the new table visualization, customer can adjust the column width as needed. Issue Resolved: opensearch-project#2212 opensearch-project#2213 opensearch-project#2305 opensearch-project#2379 opensearch-project#2579 Since this is a hookup PR, we remove un-used table vis types and options because they could be defined in vis builder. We also create follow up issues for some un-resolved PR comments. Signed-off-by: Anan Zhuang <[email protected]> * remove unused scss tyle Signed-off-by: Anan Zhuang <[email protected]> * remove total func and percentage col total func and percentage col are two features that we might need to remove or re-invent for future table vis. For hookup purpose, it doesn't make sense to include some features that we would like to remove. this PR removes total func and percentage col in both table vis and vis builder Signed-off-by: Anan Zhuang <[email protected]> * comment out cellActions currently filter in/out cell doesn't function in vis builder. we will coumment out cell actions for now. Signed-off-by: Anan Zhuang <[email protected]> Signed-off-by: Anan Zhuang <[email protected]> Signed-off-by: Ajay Gupta <[email protected]> * Security-CVEs fixes guidelines (opensearch-project#2674) * Security-CVEs fixes guidelines Signed-off-by: himsgupta1122 <[email protected]> * styling Signed-off-by: himsgupta1122 <[email protected]> * add example Signed-off-by: himsgupta1122 <[email protected]> * add documention Signed-off-by: himsgupta1122 <[email protected]> Signed-off-by: himsgupta1122 <[email protected]> Signed-off-by: Ajay Gupta <[email protected]> * [MD] Update default audit log path (opensearch-project#2793) - Fix the /tmp path issue seen on windows platform. - Change audit log to disable by default. Signed-off-by: Kristen Tian <[email protected]> Signed-off-by: Kristen Tian <[email protected]> Signed-off-by: Ajay Gupta <[email protected]> * [Vis Builder] Add field summary popovers (opensearch-project#2682) * [Vis Builder] Add field summary popovers Much of the functionality was ported from `Discover`, but largely refactored. * Add utilities to get sampled hit summaries by field * Add popover summaries * Slight refactor of special `Count` pseudofield * Use observable subscription to update sampled hits Fixes opensearch-project#950 Signed-off-by: Josh Romero <[email protected]> * [Vis Builder] Add additional unit tests Signed-off-by: Josh Romero <[email protected]> * [VisBuilder] Update naming of summary field components Signed-off-by: Josh Romero <[email protected]> * [VisBuilder] Avoid prop passing by extracting custom hooks - refactor meta field identification Signed-off-by: Josh Romero <[email protected]> * [VisBuilder] Add TODOs with issue links, fix test ID Restores previous test ID for count field button Signed-off-by: Josh Romero <[email protected]> Signed-off-by: Josh Romero <[email protected]> Signed-off-by: Ajay Gupta <[email protected]> * Bump makelogs to remove dependency on got (opensearch-project#2801) * Also dusted off the lock file a bit Fixes opensearch-project#1764 Signed-off-by: Miki <[email protected]> Signed-off-by: Miki <[email protected]> Signed-off-by: Ajay Gupta <[email protected]> * Change VisBuilder flag for docker config (opensearch-project#2804) Signed-off-by: Ashwin P Chandran <[email protected]> Signed-off-by: Ashwin P Chandran <[email protected]> Signed-off-by: Ajay Gupta <[email protected]> * addressed review comments Signed-off-by: Ajay Gupta <[email protected]> * addressed review comments Signed-off-by: Ajay Gupta <[email protected]> * addressed review comments Signed-off-by: Ajay Gupta <[email protected]> * updated unit tests Signed-off-by: Ajay Gupta <[email protected]> * dummy commit to retry build Signed-off-by: Ajay Gupta <[email protected]> * address suggestions Signed-off-by: Ajay Gupta <[email protected]> Signed-off-by: Ajay Gupta <[email protected]> Signed-off-by: abbyhu2000 <[email protected]> Signed-off-by: Miki <[email protected]> Signed-off-by: Zilong Xia <[email protected]> Signed-off-by: Craig Perkins <[email protected]> Signed-off-by: vimal K <[email protected]> Signed-off-by: Josh Romero <[email protected]> Signed-off-by: Su <[email protected]> Signed-off-by: Kristen Tian <[email protected]> Signed-off-by: Ashwin P Chandran <[email protected]> Signed-off-by: Bandini Bhopi <[email protected]> Signed-off-by: Kawika Avilla <[email protected]> Signed-off-by: Miki <[email protected]> Signed-off-by: Anan Zhuang <[email protected]> Signed-off-by: himsgupta1122 <[email protected]> Co-authored-by: Ajay Gupta <[email protected]> Co-authored-by: Qingyang(Abby) Hu <[email protected]> Co-authored-by: Miki <[email protected]> Co-authored-by: ZilongX <[email protected]> Co-authored-by: Craig Perkins <[email protected]> Co-authored-by: Vimal K <[email protected]> Co-authored-by: Josh Romero <[email protected]> Co-authored-by: Zhongnan Su <[email protected]> Co-authored-by: Kristen Tian <[email protected]> Co-authored-by: Ashwin P Chandran <[email protected]> Co-authored-by: Craig Perkins <[email protected]> Co-authored-by: Bandini <[email protected]> Co-authored-by: Kawika Avilla <[email protected]> Co-authored-by: Anan Zhuang <[email protected]> Co-authored-by: himsgupta1122 <[email protected]> Signed-off-by: Arpit Bandejiya <[email protected]>
alchemist51
force-pushed
the
feature/point-in-time
branch
from
dabc10f to
06fbeff
Compare
|
@AMoo-Miki @ananzh please review and merge this PR to update the feature branch |
AMoo-Miki
approved these changes
Mar 8, 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
[Describe what this change achieves]
Issues Resolved
[List any issues this PR will resolve]
Check List
yarn test:jestyarn test:jest_integrationyarn test:ftr