search

tolgee / tolgee-cli

The Tolgee CLI
MIT License
15 stars 11 forks source link

feat: new extractor #88

Closed stepan662 closed 2 months ago

stepan662 commented 3 months ago

Closes #63 Closes #87

OrenChapo commented 2 months ago

Thanks for your great work Stephan!

I tried the recent version of the cli with the new extractor, and it fails on "Dynamic options" for this line:

throw this.$t(
            'dialog.importUsers.badServerResponse',
            'Invalid response from Server.\nPlease retry later.',
          );

Works OK after removing the \n. Is this expected?

OrenChapo commented 2 months ago

Another issue, seems like the @tolgee-key comments ignore the specified --default-namespace parameter and assume no namespace.

stepan662 commented 2 months ago

@OrenChapo thanks for reporting, should be fixed in this PR: https://github.com/tolgee/tolgee-cli/pull/91