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

Automatic fallback to 9k6 and ECM switch #29

Merged
merged 7 commits into from
Dec 11, 2017
Merged

Conversation

icebear0815
Copy link
Contributor

Automatic fallback to 9k6 and ECM switch:
Configure seperatly the no. of retries (not dials) after the fax is send with DisableV17 = true and/or UseECM = false.

We needed the ability to switch baudrate and ECM during retries. We have had problems where the other station is receiving half pages. Lowering the baudrate fixes the problem. 99% of the documents are passing perfectly with 14k4 there is no reason in lowering ther overall baudrate.

Hopefully this patch is ok with your concept of gofaxip.

Thanks,

Lars Schindler

Automatic fallback to 9k6 and ECM switch:
Configure seperatly the no. of retries (not dials) after the fax is send with DisableV17 = true and/or UseECM = false.
Automatic fallback to 9k6 and ECM switch:
Configure seperatly the no. of retries (not dials) after the fax is send with DisableV17 = true and/or UseECM = false.
Automatic fallback to 9k6 and ECM switch
Automatic fallback to 9k6 and ECM switch:
Configure seperatly the no. of retries (not dials) after the fax is send with DisableV17 = true and/or UseECM = false.
@denzs
Copy link
Member

denzs commented Dec 4, 2017

There is a control character in config.go and an empty line with a tab in sendqfile.go
If you could fix that, i would be happy to merge your pull request :)

And please excuse the long delay!

@denzs
Copy link
Member

denzs commented Dec 8, 2017

If you want me to assist you on resolving the merge conflict, just let me know..

@icebear0815
Copy link
Contributor Author

icebear0815 commented Dec 11, 2017 via email

@denzs denzs merged commit d07d2c5 into gonicus:master Dec 11, 2017
@denzs
Copy link
Member

denzs commented Dec 11, 2017

should be fine now...
thanks a lot for your contribution! :)
btw... feel free to open another PR to add yourself to the AUTHORS file 👍

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.

2 participants