Skip to content

Release the i18next-shopify framework#970

Merged
terales merged 14 commits into
lokalise:mainfrom
Shopify:release/i18next-shopify
Jul 11, 2023
Merged

Release the i18next-shopify framework#970
terales merged 14 commits into
lokalise:mainfrom
Shopify:release/i18next-shopify

Conversation

@trishrempel

@trishrempel trishrempel commented Jul 10, 2023

Copy link
Copy Markdown
Contributor

This is a release branch for the following PRs:

Tophatting

Run the extension in debug mode and open the i18n-ally folder examples/by-frameworks/i18next-shopify.

Confirm that:

  • The extension is loaded with the i18next-shopify framework enabled:
    Screenshot 2023-07-10 at 3 38 28 PM
  • The string extraction feature works:
    Screenshot 2023-07-10 at 2 24 56 PM
  • For a pluralization string, the English text of the first pluralization sub-key is displayed, and Go To Definition works:
    Screenshot 2023-07-10 at 2 17 02 PM
  • The Usage Report lists the pluralization key as "in use":
    Screenshot 2023-07-10 at 2 19 05 PM
  • The Current File panel lists the pluralization key as "in use":
    Screenshot 2023-07-10 at 2 33 12 PM

mkevinosullivan and others added 13 commits June 21, 2023 14:10
This commit adds the `supportAutoExtraction` array of supported formats
and the `detectHardStrings` method from `GeneralFramework` to enable
hard-coded string extraction for react-i18next framework, without
needing the "general" framework to be enabled/loaded in the
`.vscode/settings.json` file.
This commit adds the i18next-shopify framework to the list of supported
frameworks. It is an extension of the React i18next framework, with some
configuration changes.

Also add i18next-shopify example app, including a pluralization example.
Derived keys (e.g., such as plurals) were being miscategorised as
missing in the usage report.  This commit checks if keys derived from
those found in the missing list (keys that are in use but considered
"not defined") are included in the list of idle keys (keys defined but
not considered in use), indicating that they are indeed defined and in
use.

Fixes lokalise#953
…ext_framework

Add extraction support to react-i18next framework
…ng_value

Display first available pluralization string if parent key of nested pluralization keys
…sted_pluralization_keys

Add support for "Go to Definition" for pluralization strings
Fixes usage report for non-missing derived keys
Derived keys (e.g., such as plurals) are being miscategorised as not
found in the Current File Panel report.  This commit checks if any
existing key paths match derived keys paths, and if so, includes them in
the list of in use key paths and removed from the "not found" list.

Fixes lokalise#959
…ort_derived_keys

Fixes Current File Panel report of not found keys
@trishrempel trishrempel marked this pull request as ready for review July 10, 2023 20:18
@trishrempel trishrempel changed the title [Draft] Release i18next-shopify framework Release the i18next-shopify framework Jul 10, 2023
Chore: fix spelling (strings, variables) within code base
@terales terales merged commit 652b827 into lokalise:main Jul 11, 2023
huacnlee pushed a commit to huacnlee/i18n-ally that referenced this pull request Aug 28, 2023
* Return first available counting value if nested

* Add extraction support to react-i18next framework

This commit adds the `supportAutoExtraction` array of supported formats
and the `detectHardStrings` method from `GeneralFramework` to enable
hard-coded string extraction for react-i18next framework, without
needing the "general" framework to be enabled/loaded in the
`.vscode/settings.json` file.

* Add i18next-shopify framework

This commit adds the i18next-shopify framework to the list of supported
frameworks. It is an extension of the React i18next framework, with some
configuration changes.

Also add i18next-shopify example app, including a pluralization example.

* Add support for Go to Definition of a translation key

* Fixes usage report for non-missing derived keys

Derived keys (e.g., such as plurals) were being miscategorised as
missing in the usage report.  This commit checks if keys derived from
those found in the missing list (keys that are in use but considered
"not defined") are included in the list of idle keys (keys defined but
not considered in use), indicating that they are indeed defined and in
use.

Fixes lokalise#953

* Fixes Current File Panel report of not found keys

Derived keys (e.g., such as plurals) are being miscategorised as not
found in the Current File Panel report.  This commit checks if any
existing key paths match derived keys paths, and if so, includes them in
the list of in use key paths and removed from the "not found" list.

Fixes lokalise#959

* Chore: fix spellings in code base

---------

Co-authored-by: Kevin O'Sullivan <mkevin.osullivan@shopify.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants