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

Unable to Access Workspace when Creating a Workspace with non-English Language #209

Closed
devleejb opened this issue Jun 21, 2024 · 9 comments · Fixed by #261
Closed

Unable to Access Workspace when Creating a Workspace with non-English Language #209

devleejb opened this issue Jun 21, 2024 · 9 comments · Fixed by #261
Assignees
Labels
bug 🐞 Something isn't working
Milestone

Comments

@devleejb
Copy link
Member

devleejb commented Jun 21, 2024

What happened:
When creating a Workspace using a non-English language, specifically when using characters outside the English alphabet, the user is unable to access the Workspace. The slug creation process seems to be encountering issues when handling non-English characters.

What you expected to happen:
I expected to be able to create a Workspace using a non-English language and access it without any issues.

How to reproduce it (as minimally and precisely as possible):

  1. Create a new Workspace.
  2. Use non-English characters (such as Korean, Chinese, etc.) during the Workspace creation process.
  3. Try to access the created Workspace.

Anything else we need to know?:
It is important to ensure that the slug creation process can handle non-English characters to allow users to create Workspaces in different languages.

Environment:

  • Operating system: ALL
  • Browser and version: ALL
  • CodePair version: ALL
@minai621
Copy link
Contributor

This issue seems to be related to #234 . Can I try this issue?

@devleejb
Copy link
Member Author

@minai621
Sure!
It would be good to leave comments on the plan before implementation.

@minai621
Copy link
Contributor

@devleejb
The root cause of the issue is that a blank string is returned because slugify does not support Korean.
slugify has no attempt to support Koren in locale. locales.json
Therefore, the solution is to use a library that romanizes multilingualism.
Among the libraries that support es6, chose transliteration, a library that seamlessly supports multilingualism.

@minai621
Copy link
Contributor

minai621 commented Jul 28, 2024

@devleejb
I tried using transliteration.
but, it can't support to all languages.
And we have to consider all the texts as well as the language.
For example, an emoji is ignored in the process of slug conversion in transliteration.
How about using Unicode, or Hash?
However, the currently created workspace needs to be converted into Unicode and Hash.

@devleejb
Copy link
Member Author

devleejb commented Jul 28, 2024

@injae-kim
Thank you for your suggestion!

I'd like to introduce only encoding because of consistency.
The introduction of Slug in CodePair was intended to quickly identify Workspaces through clean URLs (Like GitHub URL). Therefore, how about maintaining this by allowing non-English languages to be accessed through encoding, even though they may not be as clean?

@minai621
Copy link
Contributor

minai621 commented Jul 29, 2024

@devleejb
Returns an empty string if user has entered only an emoji. (ex: 😀😀😀 or unsupported languages..)
If using slug, the input must be in English or contain at least two English characters.

@devleejb
Copy link
Member Author

@minai621
I think only at least two letters is okay.
We can encode letters except English. Is it right?

@minai621
Copy link
Contributor

@devleejb

@minai621 I think only at least two letters is okay. We can encode letters except English. Is it right?

That's right. I think that's best to do it, so I made a this pull request. #261

@devleejb devleejb moved this to Backlog in CodePair Jul 31, 2024
@devleejb devleejb moved this from Backlog to In progress in CodePair Jul 31, 2024
@devleejb devleejb added this to the v0.1.1 milestone Jul 31, 2024
@github-project-automation github-project-automation bot moved this from In progress to Done in CodePair Aug 2, 2024
@github-project-automation github-project-automation bot moved this from In progress to Done in Yorkie Project - 2024 Aug 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐞 Something isn't working
Projects
Archived in project
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants