Skip to content

Add built-in Octoprint support (v2) #345

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

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

Conversation

ayufan
Copy link
Contributor

@ayufan ayufan commented Aug 6, 2021

This is based on #343. It removes a lot of limitations of #344.

This adds a built-in support for Octoprint / Klipper+Moonraker setups allowing to directly
upload and start print. It stores data in browser and sends requests directly to the install.

image

Life is great compared to previous take #344:

  • It uses a POST form, so it does not require CORS
  • It does not show a great info once uploaded, as it shows raw JSON from the other service directly in browser
  • However, it works OOB without significant extra configuration

Test it here:

@ayufan
Copy link
Contributor Author

ayufan commented Aug 8, 2021

(I noticed some problems with this. Will update comment).

It appears that:

  • it works always fine when octoprint/moonraker is on https://
  • when using http:// it works fine on Safari/Firefox which shows dialog box about unsecure connection
  • when using http://it does not send a proper file on Chrome which shows a full page prompt that you need to Send anyway

- Allow to specify used filename
- Output all settings as a comment on top of gcode
- Unify download buttons
- Allow to specify used filename
- Output all settings as a comment on top of gcode
- Unify download buttons
- Allow to specify URL/API key
- Upload&Print directly
- Document what needs to be configured
This allows to send files without having
to configure CORS.
@mikekasprzak
Copy link

Nice! I was just coming to see if anyone had added something like this (though I was checking for raw GCODE over serial).

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