Skip to content

Conversation

EkaterinePapava
Copy link

No description provided.

Copy link
Owner

@fmaclen fmaclen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for submitting this translation.
Before we roll this out we need to make a few more changes:

@fmaclen
Copy link
Owner

fmaclen commented Feb 24, 2025

Also, make sure to run npm run format and then npm run lint to make sure the linter requirements are satisfied.

@fmaclen fmaclen changed the title Adding Georgian translation feat add Georgian translation Feb 24, 2025
@fmaclen fmaclen changed the title feat add Georgian translation feat: add Georgian translation Feb 24, 2025
@EkaterinePapava
Copy link
Author

Made both changes, also ran both commands, both seems to be satisfied

Copy link
Owner

@fmaclen fmaclen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I ran the auto-formatter because there were still issues with the linter.
But at the moment the test isn't passing, check my comments below.

@EkaterinePapava
Copy link
Author

Hello,
Is anything still needed from my side?

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.

2 participants