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 the methods' doc link #1583

Merged
merged 3 commits into from Sep 6, 2018
Merged

add the methods' doc link #1583

merged 3 commits into from Sep 6, 2018

Conversation

TbhT
Copy link
Contributor

@TbhT TbhT commented Sep 3, 2018

No description provided.

TbhT and others added 2 commits September 2, 2018 19:21
add most methods' doc link and change RoomInvitation `topic`  to `roomTopic`
@TbhT
Copy link
Contributor Author

TbhT commented Sep 3, 2018

Add the corresponding doc link for most methods.

Copy link
Member

@huan huan left a comment

Choose a reason for hiding this comment

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

Awesome, thank you very much for the contributions!

@huan
Copy link
Member

huan commented Sep 3, 2018

However, it seems that this pull request is conflict with the code base

Could you please fix the conflict before we can be able to better it?

@TbhT
Copy link
Contributor Author

TbhT commented Sep 5, 2018

Seems that i have no write access to merge the PR.

@huan
Copy link
Member

huan commented Sep 5, 2018

Yes, you have no write access.

To solve the conflict, you need to sync your branch with the upstream master, and commit your latest code after you had success merged all the changes

@TbhT
Copy link
Contributor Author

TbhT commented Sep 5, 2018

Oh, i have sync the changes to latest version.

@huan huan merged commit 4265cef into wechaty:master Sep 6, 2018
@huan
Copy link
Member

huan commented Sep 6, 2018

Awesome.

Thank you very much for the contribution!

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.

None yet

2 participants