Thanks to visit codestin.com
Credit goes to github.com

Skip to content

🌐 🔠 📄 🐢 Translate docs to Emoji 🥳 🎉 💥 🤯 🤯#5385

Merged
tiangolo merged 4 commits into
fastapi:masterfrom
LeeeeT:master
Apr 1, 2023
Merged

🌐 🔠 📄 🐢 Translate docs to Emoji 🥳 🎉 💥 🤯 🤯#5385
tiangolo merged 4 commits into
fastapi:masterfrom
LeeeeT:master

Conversation

@LeeeeT

@LeeeeT LeeeeT commented Sep 13, 2022

Copy link
Copy Markdown

I just really love 💕 🤩 the idea of adding some emojis 😄😁😅 here and there across the 📄 docs 📄 (like in here for example). So I decided to write ✍ the entire documentation out of emojis 💥🎉🌟. All these words 🤮🤢 and... fancy-looking letters.. 👎👎👎 ugh... That's so boring 🥱. I believe reading documentation made of emojis will definitely (don't argue 😛🤪) be much funnier 😃 and easier 😻 for beginners. Just take a look at a new awesome 🔥 async page 🔥. So here it is, don't thank me. 💥💯💫⭐

@codecov

codecov Bot commented Sep 13, 2022

Copy link
Copy Markdown

Codecov Report

Patch and project coverage have no change.

Comparison is base (cf73051) 100.00% compared to head (3b37ad3) 100.00%.

❗ Current head 3b37ad3 differs from pull request most recent head 7b50132. Consider uploading reports for the commit 7b50132 to get more accurate results

Additional details and impacted files
@@            Coverage Diff            @@
##            master     #5385   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          540       540           
  Lines        13969     13951   -18     
=========================================
- Hits         13969     13951   -18     

see 10 files with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@github-actions

Copy link
Copy Markdown
Contributor

📝 Docs preview for commit 5590e38 at: https://6320bdb2dc53d17efc1ba804--fastapi.netlify.app

Comment thread docs/emj/docs/features.md Outdated
Comment on lines +9 to +13
* <a href="https://github.com/OAI/OpenAPI-Specification" class="external-link" target="_blank"><strong>OpenAPI</strong></a> 🏗 API, ✅ 🙂📛 <abbr title="💭 HTTP-👩‍🔬 ✅ : POST, GET, PUT, DELETE">🛠</abbr> <abbr title="💭: 🔗, 🛣, '🍵', ♒️.">🌌</abbr>, 🔢, ✔ 💪, 💂‍♂, ♒️.

@MadL1me MadL1me Sep 13, 2022

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I think "Security" in native emoji is more like "👮" than your translated "💂‍♂". However, the other parts are perfect, I like it

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.

I find amusing the seriousness of this review. 😂

@axd1x8a axd1x8a left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It looks👀 like the markup📝 is a bit🤏 broken💔, making it hard😅 to read📖 such great👍 documentation📝.
https://6320bdb2dc53d17efc1ba804--fastapi.netlify.app/emj/
image
image

@github-actions

Copy link
Copy Markdown
Contributor

📝 Docs preview for commit 3b37ad3 at: https://6321d213a4b3527ed976768f--fastapi.netlify.app

@LeeeeT

LeeeeT commented Sep 14, 2022

Copy link
Copy Markdown
Author

@feeeek 🔥🔥🔥 Fixed 🔥🔥🔥

@tiangolo

Copy link
Copy Markdown
Member

Hey there! Is this coming from a joke or something somewhere else? 😅

I imagine that given all the approvals and interactions, but I didn't see the original joke... 🤷

@axd1x8a

axd1x8a commented Sep 14, 2022

Copy link
Copy Markdown
Contributor

@feeeek 🔥🔥🔥 Fixed 🔥🔥🔥

Great💫, I think this is a very nice👍 change, making the documentation📝 much friendlier🙌. It's a pity that the previous⏮️ version of the Concurrency and async / await documentation was removed❌, I really miss it🥲.

@github-actions github-actions Bot removed the answered label Sep 14, 2022
@mrMiiao

mrMiiao commented Sep 14, 2022

Copy link
Copy Markdown

Really 😃😃😃 cool 😍😍🙂 idea 👍🧠🫦 bro 🥵😎🥵!!!

@axd1x8a

axd1x8a commented Sep 14, 2022

Copy link
Copy Markdown
Contributor

@tiangolo You see right through us. Forgive us this joke. We're just really sad about the decision to remove the emojis in #3273. They made the documentation unusual and stand out from the documentation of other frameworks. That's why we wanted to bring back that distinctive feature, so we'd be very happy if you could merge this pull request.❤️

@LeeeeT

LeeeeT commented Sep 14, 2022

Copy link
Copy Markdown
Author

I think I could do a full translation of all the pages as well as fix a few formatting issues if you would ever consider adding an emoji version of the documentation, just for fun 😊

@github-actions

Copy link
Copy Markdown
Contributor

📝 Docs preview for commit ecf3d1c at: https://639ccb4a3508160ef30a90b7--fastapi.netlify.app

@tiangolo

tiangolo commented Feb 7, 2023

Copy link
Copy Markdown
Member

So... I'm considering adding this for April Fool's Day. That's next April 1st. 🤓😂

@LeeeeT

LeeeeT commented Feb 7, 2023

Copy link
Copy Markdown
Author

Cool! 🥳 I'm going to finish up translating the remaining pages and prepare this for merging as promised. 😄

@github-actions

github-actions Bot commented Mar 8, 2023

Copy link
Copy Markdown
Contributor

📝 Docs preview for commit d51f60e at: https://64090307112b5b1b116dc96e--fastapi.netlify.app

@LeeeeT LeeeeT requested review from axd1x8a and yallxe and removed request for axd1x8a and yallxe March 8, 2023 21:58
@BrycensRanch

Copy link
Copy Markdown

LGTM

@axd1x8a

axd1x8a commented Mar 13, 2023

Copy link
Copy Markdown
Contributor

@LeeeeT, I tried to send a review, but for some reason github just gives an error, I have already contacted support, I hope they will help, but for now please wait a bit.

@LeeeeT

LeeeeT commented Mar 15, 2023

Copy link
Copy Markdown
Author

Well, @feeeek, you can submit a pull request with your proposed changes to my fork 😋

@notdanhan notdanhan left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM 👍

@baseplate-admin baseplate-admin left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@github-actions

Copy link
Copy Markdown
Contributor

📝 Docs preview for commit 24527b5 at: https://642663a0b548622487cc93a0--fastapi.netlify.app

@github-actions

Copy link
Copy Markdown
Contributor

📝 Docs preview for commit d51f60e at: https://6426a87c996bf600a71bb68f--fastapi.netlify.app

@axd1x8a

axd1x8a commented Mar 31, 2023

Copy link
Copy Markdown
Contributor

Now I know that github doesn't like this
Screenshot_20230331-133513_GitHub.png

@github-actions

Copy link
Copy Markdown
Contributor

📝 Docs preview for commit 924c274 at: https://6426a9b35d5fc100b1db0463--fastapi.netlify.app

@LeeeeT

LeeeeT commented Mar 31, 2023

Copy link
Copy Markdown
Author

@feeeek It's probably for the best... 😐

@LeeeeT LeeeeT requested a review from tiangolo April 1, 2023 07:47
@github-actions

github-actions Bot commented Apr 1, 2023

Copy link
Copy Markdown
Contributor

📝 Docs preview for commit 7b50132 at: https://6427f6b2a6d5896aca012137--fastapi.netlify.app

@tiangolo tiangolo 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.

I'm actually impressed. You did an amazing job here. The consistency of the translation from the original is great. 🤓😎


I moved the content from emj toem, that simplifies the URLs and makes using the internal scripts simpler (to run the docs live locally, etc).

em is not assigned to any other language so that's fine. https://en.wikipedia.org/wiki/List_of_ISO_639-1_codes#Table_of_all_possible_two_letter_codes


I imagine you automatized parts of it, I would love to know how you did it!


I'm also cracking up. 😂


Thank you for your contribution! 🙇🍰

@tiangolo tiangolo merged commit d4e85da into fastapi:master Apr 1, 2023
@axd1x8a

axd1x8a commented Apr 1, 2023

Copy link
Copy Markdown
Contributor

I imagine you automatized parts of it, I would love to know how you did it!


@tiangolo For all this, regexes were used to separate the text from code blocks, and then the translation through Yandex translator (link), and after all this, I fixed some inaccuracies and broken emojis. I think this can also be automated and converted to GitHub action, but we need to make it a little more stable.

@LeeeeT

LeeeeT commented Apr 1, 2023

Copy link
Copy Markdown
Author

Yea, it required some reverse engineering and use of private API in order to automate translating with Yandex. 😅 And my script for extracting readable and translatable text from md files is... well, it's better not to see it. 😹😹 Also, Yandex translator is far from ideal and some manual editing is required. So I don't think it should (and can) be automated. 🫤

Anyway, I want to thank all the participants of this pull request. ❤️ You guys made the prank even funnier. 🤪🥰

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.