opensearch-project / OpenSearch-Dashboards

📊 Open source visualization dashboards for OpenSearch.
https://opensearch.org/docs/latest/dashboards/index/
Apache License 2.0
1.69k stars 893 forks source link

[CCI] Need update dependency on `intl-messageformat-parser` to `@formatjs/icu-messageformat-parser` #3606

Open Nicksqain opened 1 year ago

Nicksqain commented 1 year ago

Describe the bug

From investigation in https://github.com/opensearch-project/OpenSearch-Dashboards/issues/3027#issuecomment-1466947789

intl-messageformat-parser has been deprecated, so we need to update it to https://www.npmjs.com/package/@formatjs/icu-messageformat-parser.

To Reproduce Steps to reproduce the behavior:

  1. Go to

    src/dev/i18n/utils/verify_icu_message.ts

  2. Scroll down to
    throw new parser.SyntaxError(
      'Unable to verify select format icu-syntax',
      'selectFormat',
      node.type,
      node.location
    );
  3. See error

OpenSearch Version 2.6.0

Dashboards Version 2.6.0

Screenshots

изображение

Host/Environment (please complete the following information):

Additional context

3027

!!! Check comment https://github.com/opensearch-project/OpenSearch-Dashboards/issues/3027#issuecomment-1466947789

joshuarrrr commented 1 year ago

@Nicksqain I updated this issue to focus on the dependency update/replacement we want to do here.

joshuarrrr commented 1 year ago

@Nicksqain Do you want this issue assigned, or should I mark it good first issue for someone else to pick up?

Nicksqain commented 1 year ago

@Nicksqain Do you want this issue assigned, or should I mark it good first issue for someone else to pick up?

@joshuarrrr Yes, assign me please, I want to know how it's done.