Skip to content

feat: translate testing-environments (done) #182

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

Merged
merged 6 commits into from
Sep 16, 2021

Conversation

danglequocbao001
Copy link
Contributor

Feat/translate addons

@danglequocbao001 danglequocbao001 changed the title Merge pull request #1 from danglequocbao001/feat/translate-addons chore: pass commit from my folked repo Sep 15, 2021
@danglequocbao001 danglequocbao001 changed the title chore: pass commit from my folked repo feat: pass commit (translate addons) from my folked repo Sep 15, 2021
@netlify
Copy link

netlify bot commented Sep 15, 2021

✔️ Deploy Preview for vi-reactjs-org ready!

🔨 Explore the source changes: f06e669

🔍 Inspect the deploy log: https://app.netlify.com/sites/vi-reactjs-org/deploys/6141ec3f49ae0b0007101242

😎 Browse the preview: https://deploy-preview-182--vi-reactjs-org.netlify.app

@danglequocbao001
Copy link
Contributor Author

danglequocbao001 commented Sep 15, 2021

@Ryan-LT giúp mình review pr này với nha, mình mới làm lại flow merge cho đúng á (my-repo-branch -> my-repo-main -> pull request -> reactjs-repo-main, bạn có thể review ở đây danglequocbao001@1c3b0c8 , cảm ơn nhiều!!

@Ryan-LT
Copy link
Member

Ryan-LT commented Sep 15, 2021

@Ryan-LT giúp mình review pr này với nha, mình mới làm lại flow merge cho đúng á (my-repo-branch -> my-repo-main -> pull request -> reactjs-repo-main, bạn có thể review ở đây danglequocbao001@1c3b0c8 , cảm ơn nhiều!!

Ok, Nhưng sao mình ko thấy có thay đổi gì mới hết vậy nhỉ?

@danglequocbao001
Copy link
Contributor Author

@Ryan-LT giúp mình review pr này với nha, mình mới làm lại flow merge cho đúng á (my-repo-branch -> my-repo-main -> pull request -> reactjs-repo-main, bạn có thể review ở đây danglequocbao001@1c3b0c8 , cảm ơn nhiều!!

Ok, Nhưng sao mình ko thấy có thay đổi gì mới hết vậy nhỉ?

Trước đây mình làm sai thao tác, từ branch ở repo mình merge thẳng vô main của reactjs luôn, nên mình muốn merge lại để có thể tự kiểm soát commit nhằm tránh conflict á.

@Ryan-LT
Copy link
Member

Ryan-LT commented Sep 15, 2021

Trước đây mình làm sai thao tác, từ branch ở repo mình merge thẳng vô main của reactjs luôn, nên mình muốn merge lại để có thể tự kiểm soát commit nhằm tránh conflict á.

Bạn có thể fork lại và clone lại từ reactjs main ko? Vì merge như vầy mình sợ sẽ override vài file có sẵn mất.

@danglequocbao001
Copy link
Contributor Author

Trước đây mình làm sai thao tác, từ branch ở repo mình merge thẳng vô main của reactjs luôn, nên mình muốn merge lại để có thể tự kiểm soát commit nhằm tránh conflict á.

Bạn có thể fork lại và clone lại từ reactjs main ko? Vì merge như vầy mình sợ sẽ override vài file có sẵn mất.

Vậy mình sẽ contribute vô thêm trong pr này luôn rồi bạn review và merge nha, giữ pr này giúp mình nhé, bạn thấy vậy có ổn không?

@thinhlesdev
Copy link

Vậy mình sẽ contribute vô thêm trong pr này luôn rồi bạn review và merge nha, giữ pr này giúp mình nhé, bạn thấy vậy có ổn không?

Okay! 4 chân luôn bạn :D

@danglequocbao001
Copy link
Contributor Author

Vậy mình sẽ contribute vô thêm trong pr này luôn rồi bạn review và merge nha, giữ pr này giúp mình nhé, bạn thấy vậy có ổn không?

Okay! 4 chân luôn bạn :D

Okay, hứa sẽ pull check cẩn thận đảm bảo không ghi đè file :v

@danglequocbao001 danglequocbao001 changed the title feat: pass commit (translate addons) from my folked repo feat: translate testing-environments Sep 15, 2021
@danglequocbao001 danglequocbao001 changed the title feat: translate testing-environments feat: translate testing-environments (not done) Sep 15, 2021
@Ryan-LT Ryan-LT added the WIP Work in progress label Sep 15, 2021
chore: pull code from global to folked repo
@danglequocbao001 danglequocbao001 changed the title feat: translate testing-environments (not done) feat: translate testing-environments (done) Sep 15, 2021
@Ryan-LT Ryan-LT removed the WIP Work in progress label Sep 15, 2021
@Ryan-LT Ryan-LT mentioned this pull request Sep 16, 2021
95 tasks
Copy link
Member

@Ryan-LT Ryan-LT left a comment

Choose a reason for hiding this comment

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

Better over time ⚡

@Ryan-LT Ryan-LT merged commit 81947c4 into reactjs:main Sep 16, 2021
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.

3 participants