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

feat!: upgrade default mysql charset and collation to utf8mb4 #1084

Merged

Conversation

Danyal-Faheem
Copy link
Contributor

fixes #938.

Changes

  • Changes the default character set of MySQL to utf8mb4 and the default collation to utf8mb4_unicode_ci.
  • Also updated the dosqlshell command to use utf8mb4 as the default character set.

Testing

  • I have tested these changes on tutor dev and tutor K8s. I followed these steps:
    • Ran tutor [dev|k8s] launch
    • Imported the demo course and made sure it was working fine
    • Created admin and student users successfully
    • Created a new course with emojis in the title and in subsection titles and made sure changes were being published and reflected in the LMS

Future Work

A new do command tutor local do change-charset-collation to allow users to upgrade the charset and collation of their previous tutor installations will be added in a separate PR. A WIP PR is already up at #1079.

@regisb
Copy link
Contributor

regisb commented Jun 25, 2024

What? I thought that this upgrade was already part of the redwood branch. If that's not the case then we need to release 18.1.0 ASAP.

@Danyal-Faheem
Copy link
Contributor Author

@regisb, the original PR #1065 was closed without being merged so the changes were not reflected in 18.0.0.

@regisb regisb merged commit c4fd306 into overhangio:master Jun 25, 2024
2 checks passed
@regisb regisb deleted the danyalfaheem/upgrade-mysql-to-utf8mb4 branch June 25, 2024 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

We must migrate the mysql databases from utf8mb3 to utf8mb4
3 participants