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.

Sign up for GitHub

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

Jump to bottom

Crash with cells with too much text - unknown limit...? #75

Closed
Fabiombolo opened this issue May 19, 2023 · 1 comment · Fixed by #90
Closed

Crash with cells with too much text - unknown limit...? #75

Fabiombolo opened this issue May 19, 2023 · 1 comment · Fixed by #90
Assignees
@ttsukagoshi
Labels
enhancement New feature or request

Comments

@Fabiombolo
Copy link

Got an error when there is too much text to translate in one cell, but limit is unknown to the user
@ttsukagoshi
Copy link
Owner

@Fabiombolo Thanks for the heads-up, that would be a great idea💡 I'll work on it.

This limitation comes from the Google Apps Script's 2KB cap on the URL length in UrlFetchApp.fetch, in this case calling the DeepL API.

@ttsukagoshi ttsukagoshi self-assigned this May 20, 2023
@ttsukagoshi ttsukagoshi added the enhancement New feature or request label May 20, 2023
ttsukagoshi added a commit that referenced this issue Jun 3, 2023
@ttsukagoshi
Update error message for too much text
0773eb9 
to show a rough estimate of the optimal text length #75
@ttsukagoshi ttsukagoshi mentioned this issue Jun 3, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ttsukagoshi ttsukagoshi

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update error message for cells with too much text ttsukagoshi/sheetsL
2 participants
@Fabiombolo @ttsukagoshi