-
Notifications
You must be signed in to change notification settings - Fork 59
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
Crash when trying to export a chat. #104
Comments
I do no see any way to adjust the encoding. |
That's a funny one, will look into it asap. |
Is it possible that your terminal operates in Windows-1250 encoding? |
I don't know how to check . It has no problem with emojis so I think it
should be utf-8...
I am using Windows terminál running PowerShell.
Dne út 27. 8. 2024 21:27 uživatel Yiorgis Gozadinos <
***@***.***> napsal:
… Is it possible that your terminal operates in Windows-1250 encoding?
—
Reply to this email directly, view it on GitHub
<#104 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AEN2KDZ2EKVHNHGRALISVVDZTTHJPAVCNFSM6AAAAABNC2EBGKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMJTGM2DIOJUHE>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Would you be so kind and tell me what the following reports in your python? import locale
locale.getencoding() If it is not |
‚cp1250‘
From: Yiorgis Gozadinos ***@***.***>
Sent: Tuesday, August 27, 2024 9:55 PM
To: ggozad/oterm ***@***.***>
Cc: mcDandy ***@***.***>; Author ***@***.***>
Subject: Re: [ggozad/oterm] Crash when trying to export a chat. (Issue #104)
Would you be so kind and tell me what the following reports in your python?
import locale
locale.getencoding()
If it is not UTF-8 would you be able to run from PR #105 <#105> which should fix it for you?
—
Reply to this email directly, view it on GitHub <#104 (comment)> , or unsubscribe <https://github.com/notifications/unsubscribe-auth/AEN2KDZZCLDPACW6RPPBVI3ZTTKSDAVCNFSM6AAAAABNC2EBGKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMJTGM4TCNJQHA> .
You are receiving this because you authored the thread. <https://github.com/notifications/beacon/AEN2KD45PINBYBISNI6YILLZTTKSDA5CNFSM6AAAAABNC2EBGKWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTUJ4OGZI.gif> Message ID: ***@***.*** ***@***.***> >
|
Right, so like I said your environment is configured with windows1250. I didn't know windows does that in certain countries... Are you able to verify that #105 solves the problem? You can use
to install it in a virtual env and try it out. |
Looks like it is working.
From: Yiorgis Gozadinos ***@***.***>
Sent: Tuesday, August 27, 2024 9:55 PM
To: ggozad/oterm ***@***.***>
Cc: mcDandy ***@***.***>; Author ***@***.***>
Subject: Re: [ggozad/oterm] Crash when trying to export a chat. (Issue #104)
Would you be so kind and tell me what the following reports in your python?
import locale
locale.getencoding()
If it is not UTF-8 would you be able to run from PR #105 <#105> which should fix it for you?
—
Reply to this email directly, view it on GitHub <#104 (comment)> , or unsubscribe <https://github.com/notifications/unsubscribe-auth/AEN2KDZZCLDPACW6RPPBVI3ZTTKSDAVCNFSM6AAAAABNC2EBGKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMJTGM4TCNJQHA> .
You are receiving this because you authored the thread. <https://github.com/notifications/beacon/AEN2KD45PINBYBISNI6YILLZTTKSDA5CNFSM6AAAAABNC2EBGKWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTUJ4OGZI.gif> Message ID: ***@***.*** ***@***.***> >
|
Excellent, thank you for the bug submission and the testing! Will release asap. |
Just noticed that you gave me a command to do it. I manually edited the offending file 😅.
From: Yiorgis Gozadinos ***@***.***>
Sent: Tuesday, August 27, 2024 11:00 PM
To: ggozad/oterm ***@***.***>
Cc: mcDandy ***@***.***>; Author ***@***.***>
Subject: Re: [ggozad/oterm] Crash when trying to export a chat. (Issue #104)
‚cp1250‘
Right, so like I said your environment is condigured with windows1250. I didn't know windows does that in certain countries...
Are you able to verify that #105 <#105> solves the problem? You can use
pip install ***@***.***/pull/105/merge
to install it in a virtual env and try it out.
—
Reply to this email directly, view it on GitHub <#104 (comment)> , or unsubscribe <https://github.com/notifications/unsubscribe-auth/AEN2KD6OBRG7D3UHP2WA6XTZTTSG3AVCNFSM6AAAAABNC2EBGKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMJTGUZTCMZQHE> .
You are receiving this because you authored the thread. <https://github.com/notifications/beacon/AEN2KDZJPD7332CW5YBSKVDZTTSG3A5CNFSM6AAAAABNC2EBGKWGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTUJ4WX22.gif> Message ID: ***@***.*** ***@***.***> >
|
Force utf-8 when exporting messages to a file. Closes #104.
I made a story (a wierd one) but when trying to export it oterm crashed.
The text was updated successfully, but these errors were encountered: