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

Begin localization efforts #12

Draft
wants to merge 16 commits into
base: main
Choose a base branch
from
Draft

Begin localization efforts #12

wants to merge 16 commits into from

Conversation

CenTdemeern1
Copy link
Member

@CenTdemeern1 CenTdemeern1 commented Jun 10, 2023

THIS IS A DRAFT.

It should not be merged (or marked as ready for review) in its current state.

I may or may not have found the wrong library on Google1 (so not the one that was referred to in #102). Is this library fine, or should I switch what library is used?

According to the docs of the library mentioned in #10, its SvelteKit solution is inspired by the library I'm using here.3

Footnotes

  1. Update: nope, I completely blame @DaniruKun
    Screenshot 2023-06-10 at 02 52 28

  2. The library being referred to here is https://github.com/kaisermann/svelte-i18n. For reference, the library being used right now is https://github.com/sveltekit-i18n/lib.

  3. See https://github.com/kaisermann/svelte-i18n/blob/main/docs/Svelte-Kit.md#usage-in-svelte-kit

I may or may not have found the wrong library on Google. Will discuss this in the PR.
@netlify
Copy link

netlify bot commented Jun 10, 2023

Deploy Preview for melodic-snickerdoodle-1f6e46 ready!

Name Link
🔨 Latest commit f3cafd9
🔍 Latest deploy log https://app.netlify.com/sites/melodic-snickerdoodle-1f6e46/deploys/649df450ca73370008bf4217
😎 Deploy Preview https://deploy-preview-12--melodic-snickerdoodle-1f6e46.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@DaniruKun
Copy link
Member

@CenTdemeern1 I think the sveltekit-i18n/lib library is ok, I do prefer that it has fewer dependencies. And this will also save time on reworking more stuff. In this PR let's just finish the core setup for at minimum en.

src/lib/lang/index.json Outdated Show resolved Hide resolved
src/lib/translations.js Outdated Show resolved Hide resolved
CenTdemeern1 and others added 3 commits June 10, 2023 18:51
Got stuck in debugging for a bit, misread an error

Co-Authored-By: Vladyslav Tatarin <[email protected]>
@CenTdemeern1 CenTdemeern1 self-assigned this Jun 10, 2023
Dedupes a bunch of code.
I didn't want this to begin with, but I just didn't know how to do it.
Skill issue on my end, apologies.
How did this happen
@CenTdemeern1
Copy link
Member Author

CenTdemeern1 commented Jun 18, 2023

Hey @DaniruKun apparently marked (the markdown package we're using) requires Node 18 or up, should CI be updated? (or can we only use 16 for some reason)

Show logs
3:51:06 AM: npm ERR! notsup Not compatible with your version of node/npm: [email protected]
3:51:06 AM: npm ERR! notsup Required: {"node":">= 18"}
3:51:06 AM: Failed during stage 'Install dependencies': dependency_installation script returned non-zero exit code: 1
3:51:06 AM: npm ERR! notsup Actual:   {"npm":"8.19.4","node":"v16.20.0"}

@DaniruKun
Copy link
Member

Should be possible to set 18 in Netlify, I'll let you know when it's ready.

@DaniruKun
Copy link
Member

@CenTdemeern1 updated Netlify builder node version to latest 18 minor-patch ver.

@DaniruKun DaniruKun added the enhancement New feature or request label Jun 18, 2023
@DaniruKun DaniruKun linked an issue Jun 18, 2023 that may be closed by this pull request
@CenTdemeern1
Copy link
Member Author

Alrighty let's try a rerun

@CenTdemeern1
Copy link
Member Author

I'll commit... uh... something

@CenTdemeern1
Copy link
Member Author

"this branch has not been deployed" ?????

@DaniruKun
Copy link
Member

Deploy should now be live at https://main--melodic-snickerdoodle-1f6e46.netlify.app
Cleared the CI cache, maybe it was not happy about the Node change at first.

@CenTdemeern1
Copy link
Member Author

Looks like it works, thanks!

@DaniruKun
Copy link
Member

@CenTdemeern1 what's the current status of the PR btw? Just want to know somewhere during the week so I know if I should create a new branch off of it, or we merge it into the trunk, since I want to work on the redesign a bit.

@CenTdemeern1
Copy link
Member Author

I'll continue working on this when I'm done with GMTK, this branch should be able to be merged soon

@DaniruKun
Copy link
Member

@CenTdemeern1 do you need some support in this branch atm?

@CenTdemeern1
Copy link
Member Author

@DaniruKun since Haato's diary is finished now I can get back to doing this, so help is not needed but if you want to help sure

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: 🏗 In progress
Development

Successfully merging this pull request may close these issues.

Add localisation
3 participants