Skip to content

Conversation

@taooceros
Copy link
Member

No description provided.

@taooceros taooceros marked this pull request as draft October 14, 2020 03:12
@jjw24
Copy link
Member

jjw24 commented Oct 14, 2020

Hey I just saw the PR is going against Master. Can you change the base to Dev instead please, you can do this by clicking on 'Edit'. We develop against the Dev branch.

image

@jjw24 jjw24 added the bug Something isn't working label Oct 14, 2020
@taooceros taooceros changed the base branch from master to dev October 14, 2020 10:16
@taooceros
Copy link
Member Author

Yes, thank you for telling me that I can do this.

@jjw24
Copy link
Member

jjw24 commented Oct 15, 2020

Hey @taooceros , it would be better to separate out the two change set please, otherwise one may delay the other due to review/discussion. Let me know if you need help to do this.

Once the improvements from bao-qian is in separate pr, i can review it, as I have a few questions about the change set such as why remove translate as it sets the future use for translating other languages, and that particular line removed is currently used to translate user's Chinese into Pinyin so the string matcher can compare.

@taooceros
Copy link
Member Author

Sorry, I do want to separate them into two branches... . However, it seems that they have automatically merge each other. I will take a look on them and try to seperate them into two pull requests later.
Thanks for telling me that they are being together!

@taooceros taooceros force-pushed the master branch 2 times, most recently from 424d757 to 135f63a Compare October 16, 2020 16:15
@taooceros
Copy link
Member Author

I will create a new pull request from another branch regarding that this pull request is based on the master branch, while I would like to keep it the same as the main repository of Flow.

@taooceros taooceros closed this Oct 17, 2020
@taooceros
Copy link
Member Author

Check out #179 and #178

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants