-
Notifications
You must be signed in to change notification settings - Fork 112
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
Conversation
Feat/translate addons
✔️ 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 |
@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 á. |
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? |
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 |
chore: pull code
chore: pull code from global to folked repo
….org into feat/translate-addons
Feat/translate addons
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Better over time ⚡
Feat/translate addons