-
Notifications
You must be signed in to change notification settings - Fork 12
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
Support for Valetudo RE #9
Comments
Sorry, this is out of scope for this project. I would not even accept a ready-to-merge pull request. I do not want to encourage the usage of Valetudo RE in any way, shape or form. There will be no further discussion. |
@ccoors thank you for your response! No problem, sure, your project your rules ;) |
@ccoors may I ask what you reasons are for not encouraging the use of RE? |
Apart from my personal feelings for RE, which I won't go into (there has been enough talk about that on various channels), adding support for RE would require a lot of work since the old API works quite differently. |
This really sound like "I don't want, becaus I don't want." Well, thats bad, so the only hope is that someome will fork it. |
It would be very kind if the Valetudo RE could be also supported one day.
Currently I got the following error after having Test Connection button pressed:
The text was updated successfully, but these errors were encountered: