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

Unfinished updates for new Facebook API #10

Open
wants to merge 43 commits into
base: master
Choose a base branch
from

Conversation

Kreastr
Copy link

@Kreastr Kreastr commented Jun 1, 2018

Message forwarding from FB to Telegram works
Message forwarding from Telegram to FB works with defects (messages sometimes are forwarded to wrong person)

@Liryna
Copy link
Owner

Liryna commented Jun 5, 2018

Hi @Kreastr ,

Thank you for your pull request.

Can you make a clear history of you git commit to see clearly the changes happening here ?

@rschaerer
Copy link

Hi @Kreastr , you did some hard work. Thanks for this. However I also agree with @Liryna that a clear commit history might speed up the review process (or even make a review possible).

Feel free to ping me if you need further assistance.

I know that this demand for more work from you can be frustrating but hey: let's do it for the quality. ❤️

Cheers
@rschaerer

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

5 participants