Skip to content
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

Failed to download Multilingual file translations with the dest parameter #486

Open
andrii-bodnar opened this issue Oct 4, 2022 · 4 comments

Comments

@andrii-bodnar
Copy link
Member

Describe the bug

The dest parameter is not working for the multicolumn spreadsheet translations downloading.

To Reproduce

Steps to reproduce the behavior:

  1. crowdin.yml configuration file:
"project_id" : "-ID-"
"api_token" : "-TOKEN-" 
"base_path" : "."
"base_url" : "https://crowdin.com"

"preserve_hierarchy": true

files: [ 
 {
    "source": "strings.csv",
    "dest": "dir/strings.csv",
    "translation": "strings.csv",
    "first_line_contains_header": true,
    "scheme": "identifier,source_phrase,de,es-ES,fr,ja,ko,pt-PT,zh-CN"
  }
]
  1. The strings.csv file content:
"","source_phrase","de","es","fr","ja","ko","pt","zh"
"identifier.1","test string 1","","","","","","",""
"identifier.2","test string 2","","","","","","",""
"identifier.3","test string 3","","","","","","",""
"identifier.4","test string 4","","","","","","",""
  1. Execute CLI command: crowdin push (to upload source file to the Crowdin project)
  2. Execute CLI command: crowdin pull
  3. See an error

If you remove the dest parameter - everything is working as expected (requires sources reupload)

Expected behavior

Successfully downloaded translations.

Environment:

  • OS: MacOS, Ubuntu, Windows
  • Version: CLI latest version

Screenshots

image

Additional context

The dest option description
Multicolumn Spreadsheet

@andrii-bodnar andrii-bodnar added bug help wanted hacktoberfest This issue welcomes contributions for Hacktoberfest labels Oct 4, 2022
@andrii-bodnar andrii-bodnar removed the hacktoberfest This issue welcomes contributions for Hacktoberfest label Nov 25, 2022
@ddninja
Copy link
Contributor

ddninja commented Oct 24, 2023

hi @andrii-bodnar , I would like to work on this issue, can you please assign this to me?

@ddninja
Copy link
Contributor

ddninja commented Oct 24, 2023

@andrii-bodnar i was not able to reproduce the error with the above steps.
Sharing you the screenshot. I am getting same error in when trying to pull, irrespective of whether dest is there in the yaml file or not. Can you please check if the issue is still there ?
image

@andrii-bodnar
Copy link
Member Author

Hi @ddninja, I just tried and I was able to reproduce this.

Please make sure you're using the crowdin.yml configuration from the issue description. As I can see, your crowdin push output is different from the output in the description (the dir directory wasn't created in the Crowdin project)

@andrii-bodnar
Copy link
Member Author

@andrii-bodnar andrii-bodnar changed the title Failed to download multicolumn spreadsheet file translations with the dest parameter Failed to download Multilingual file translations with the dest parameter Oct 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants