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

Update README.md #1103

Merged
merged 4 commits into from
Sep 26, 2020
Merged

Update README.md #1103

merged 4 commits into from
Sep 26, 2020

Conversation

oldman4U
Copy link
Contributor

Add information about config.ini and that it has to be updated together with the firmware and the TFT folder

@Acenotass
Copy link
Contributor

@oldman4U
It makes sense to specify that the config.ini file must contain UTF encoding.
Otherwise, the firmware freezes, I had it when they made menu items in Russian, but saved it in windows 1251 encoding.

@oldman4U
Copy link
Contributor Author

Hi.

UTF or should I mention a specific encoding (8/16/32)?

@Acenotass
Copy link
Contributor

i think utf16, for chinese support, but for russian utf8 is enough

@oldman4U
Copy link
Contributor Author

Thank you Acenotass!

Update is done.

@oldman4U
Copy link
Contributor Author

Q: Can I add a Linked Issue to my own PR? Somehow this seems to be deactivated.

THX

@radek8
Copy link
Contributor

radek8 commented Sep 19, 2020

Write in the description
Resolves #1095

@oldman4U
Copy link
Contributor Author

Thank you radek8

@traffic-light
Copy link
Contributor

@bigtreetech when do you guys merge this? This it's pretty usefull because the readme is outdated

@oldman4U
Copy link
Contributor Author

This is especially strange because they merged only one PR...

added the information how the update looks on TFT
@bigtreetech bigtreetech merged commit 2ad15bd into bigtreetech:master Sep 26, 2020
Manu512 pushed a commit to Manu512/BIGTREETECH-TouchScreenFirmware that referenced this pull request Nov 3, 2020
jeffeb3 pushed a commit to V1EngineeringInc/BIGTREETECH-TouchScreenFirmware that referenced this pull request Nov 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants