Skip to content

feat: add swedish translation site - #50

Open
MaximeKoitsalu wants to merge 13 commits into
turkupy:devfrom
MaximeKoitsalu:swedish-translation-site
Open

feat: add swedish translation site#50
MaximeKoitsalu wants to merge 13 commits into
turkupy:devfrom
MaximeKoitsalu:swedish-translation-site

Conversation

@MaximeKoitsalu

@MaximeKoitsalu MaximeKoitsalu commented Oct 3, 2020

Copy link
Copy Markdown

update translations.json file with swedish translations and added missing frontpage.date and frontpage.hosted-by keys to event component

Refs: #31

@MaximeKoitsalu MaximeKoitsalu changed the title Swedish translation site feat: add swedish translation site Oct 3, 2020

@magdapoppins magdapoppins left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Some minor typos, but otherwise looks good! 🤩

## Summary
During April to June, we managed to have study circles for each chapter!
## Sammanfattning
Mellan april och juni, lyckades vi ha studeringscirklar för varje vakpitel!

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Typo in kapitel 👀

"en": "Turku.py is a programming community for women and non-binaries in Turku, Finland. We organize meetups, workshops and afterworks that aim to support the local tech culture, share skills and knowlegde and inspire people from minority groups in the fields of programming, devops and data science.",
"fi": "",
"sv": ""
"sv": "Turku.py är ett programmerings community för kvinnor och icke-binära i Åba, Finland. Vi anordnar träffar, workshops och afterworks med målet att stödja den lokala tech-kulturen, dela kunskap och kompetenser samt inspirera människor från minoritetsgrupper inom områdena programmering, devops och datavetenskap."

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Åbo 😜

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Haha, oops!

"sv": "Datum"
},
"frontpage.hosted-by": {
"en": "Host by",

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hosted by, maybe "Arrangerat av"?

@MaximeKoitsalu MaximeKoitsalu Oct 5, 2020

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Sounds good!

@magdapoppins

Copy link
Copy Markdown
Member

@MaximeKoitsalu I can see you have some changes that happen in both pull requests, could you separate the two (translations of the .md files in one pr and some site translations in the other)? Maybe this was a branch-glitch?

@MaximeKoitsalu

Copy link
Copy Markdown
Author

eebe489 has the changes you requested and af1aef8 aims to remove the translations of the event index.md files

@magdapoppins

Copy link
Copy Markdown
Member

eebe489 has the changes you requested and af1aef8 aims to remove the translations of the event index.md files

Does this mean we can close the other pull request, since the same things are fixed in this? In that case, could you resolve the conflicts and I can approve and merge this pr and close the other one? ✅

@MaximeKoitsalu

Copy link
Copy Markdown
Author

eebe489 has the changes you requested and af1aef8 aims to remove the translations of the event index.md files

Does this mean we can close the other pull request, since the same things are fixed in this? In that case, could you resolve the conflicts and I can approve and merge this pr and close the other one? ✅

You should be able to merge both. One pull request only adds the missing sv *.md files and the other pull request focuses on the translation of the website itself.

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