Skip to content
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

Add register event && Add friendship && Add FileBox #20

Merged
merged 10 commits into from
Apr 13, 2020
Merged

Add register event && Add friendship && Add FileBox #20

merged 10 commits into from
Apr 13, 2020

Conversation

dchaofei
Copy link
Collaborator

@dchaofei dchaofei commented Apr 7, 2020

Add register event.
Add friendship.
Add FileBox.

In order to solve the circular reference, I newly added the impl directory as the implementation of the interface, and write the interface to the wechaty directory. This should not be the best solution, but it can solve the current problem.

@dchaofei dchaofei requested review from huan, dingdayu, a team and SilkageNet and removed request for a team April 7, 2020 05:10
@dchaofei dchaofei changed the title Add event Add register event && Add friendship Apr 7, 2020
wechaty/impl/wechaty.go Outdated Show resolved Hide resolved
@dchaofei dchaofei changed the title Add register event && Add friendship Add register event && Add friendship && Add FileBox Apr 9, 2020
@huan
Copy link
Member

huan commented Apr 10, 2020

Great to see all of us approved this PR to be merged!

I'd like to suggest that:

  1. Please feel free to merge a PR after it has been approved and you believe everything is OK with it.
  2. The best practice is to split the features to separate PRs instead put everything into one. So please create different branches for each feature, so that we can make it easy to discuss about it in different threads (Issues/PRs).

Thank you very much, I hope you all have a great weekend!

@dchaofei
Copy link
Collaborator Author

Can this pr be merged

@huan
Copy link
Member

huan commented Apr 13, 2020

@dchaofei Of course, please feel free to help yourself to merge this pull request because we have already two approvements on it!

@dchaofei
Copy link
Collaborator Author

I don't have merge permission. 😊

@huan
Copy link
Member

huan commented Apr 13, 2020

Perhaps GitHub does not permit the PR author to merge the PR by himself?

Ok, I'll merge this for you, and we can ask other contributors who have the write permission to help to merge in the future!

@huan huan merged commit 53c28d0 into wechaty:master Apr 13, 2020
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.

4 participants