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

Maintenance #1

Closed
wants to merge 704 commits into from
Closed

Maintenance #1

wants to merge 704 commits into from

Conversation

NilsRo
Copy link
Owner

@NilsRo NilsRo commented Feb 28, 2022

  • Your changes are not possible to do through a plugin and relevant
    to a large audience (ideally all users of OctoPrint)
  • If your changes are large or otherwise disruptive: You have
    made sure your changes don't interfere with current development by
    talking it through with the maintainers, e.g. through a
    Brainstorming ticket
  • Your PR targets OctoPrint's devel branch if it's a completely
    new feature, or maintenance if it's a bug fix or improvement of
    existing functionality for the current stable version (no PRs
    against master or anything else please)
  • Your PR was opened from a custom branch on your repository
    (no PRs from your version of master, maintenance or devel please),
    e.g. dev/my_new_feature or fix/my_bugfix
  • Your PR only contains relevant changes: no unrelated files,
    no dead code, ideally only one commit - rebase and squash your PR
    if necessary!
  • Your changes follow the existing coding style
  • If your changes include style sheets: You have modified the
    .less source files, not the .css files (those are generated with
    lessc)
  • You have tested your changes (please state how!) - ideally you
    have added unit tests
  • You have run the existing unit tests against your changes and
    nothing broke
  • You have added yourself to the AUTHORS.md file :)

What does this PR do and why is it necessary?

How was it tested? How can it be tested by the reviewer?

Any background context you want to provide?

What are the relevant tickets if any?

Screenshots (if appropriate)

Further notes

foosel and others added 30 commits May 4, 2023 17:34
* Handles M115 GEOMETRY capability report

* No need to use a separate format string...

* Origin has no effect on machine height...

* ♻️ virtual: refactor area report

Now depends on a single config setting `m115ReportArea`.

If true, the virtual printer will generate an area report from the current
printer profile and include it in the M115 reponse. Ships as false
by default.

---------

Co-authored-by: Gina Häußge <gina@octoprint.org>
It was not storing its current value properly, thanks to an incomplete
refactoring ages ago during which etag and lastmodified types were
joined into httpheader.

Closes #4806
It was completely non deterministic due to no info about last print
and upload date having been collected in the backend.

This is fixed now, making storing by last upload date, last print
etc also apply to folders.

The corresponding data has also been made available in the frontend.

Closes #4800
`None` values were converted to an empty string by sv, leading to
comparison issues with (correct) integers. Using the default order
value for sv fixes that.

Closes #4818
Also improve error messaging in that regard (and fix some typo
issues in an existing FAQ URL).

Closes #4819
* improve pause gcode

* linting

* comment and improve description

* typo
Thanks for the (unintentional) heads-up, @cp2004
Co-authored-by: Gina Häußge <gina@octoprint.org>
* log extruder and file position on pause

* satisfy linter

* move values into event, include also cancel action

* please the linter

* updated event documentation

* ♻️ Remove data redundancy

We really don't want to include origin and filename
in the event again when all we actually want
is the fileposition.

---------

Co-authored-by: Gina Häußge <gina@octoprint.org>
Needs to take the current base url and possible
path prefixes into account.

Closes #4829
* provide variables from multiple plugins

* 🎨 Fix linter issues

---------

Co-authored-by: Gina Häußge <gina@octoprint.org>

Closes #4756
We need to reset emptyLayers (and should - for the sake of consistency -
also reset percentageByLayer) after running an analysis because
otherwise old values will stay in there until page reload, causing layers
to not render that actually are not empty but were in any previously
loaded model.
foosel and others added 29 commits February 7, 2024 14:46
Get rid of outdated vendored imp.py. Also fixes editable plugin installs
under current tooling.
Should make unit tests happy again.
3.12 seems to be significantly faster than 3.7, let's utilize that!
They are causing issues with third party plugins, and the
proper roles and labels should provider the same and
hopefully even better functionality.

Update of #4928, see also LazeMSS/OctoPrint-UICustomizer#315
Will contain the base version (so 1.11.0 in case of 1.11.0.dev,
1.11.0rc* etc)
Should reduce the annoyance of the reload overlay caused by
settings updates in the background.
We so far only ever set it on server connect, which would cause
a reload overlay to trigger on every reconnect if the settings
had been changed in any way since the initial connect.

This should prevent this in most cases.
Ensure reauthentication won't be requested for users that are
authenticated through remote user headers on the incoming
requests.

Also refactored login mechanism handling a bit.

Closes #4966
Empty filters would filter everything, that was certainly not
intended.

Closes #4967
It required checking the warnings and funding calls even if
they weren't present. This is now checked first.

Closes #4969
Should reduce the annoyance of the reload overlay caused by
settings updates in the background.

It should also help with the reload prompts sometimes observed
during reauthentication.
@NilsRo NilsRo closed this Mar 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet