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

[BUG] Hide Succesful printed files hides folders also #5021

Closed
1 of 4 tasks
notoriouscarl opened this issue May 23, 2024 · 2 comments
Closed
1 of 4 tasks

[BUG] Hide Succesful printed files hides folders also #5021

notoriouscarl opened this issue May 23, 2024 · 2 comments
Labels
bug Issue describes a bug done Done but not yet released
Milestone

Comments

@notoriouscarl
Copy link

notoriouscarl commented May 23, 2024

octoprint-systeminfo-20240523190900.zip

The problem

Hide successful printed files hides folders also. while enabled, it will only show folders that have NO successful printed files. even if the folder has un-printed or unsuccessful prints in the folder the folder is hidden.

Did the issue persist even in safe mode?

Yes, it did persist

If you could not test in safe mode, please state why ("currently printing" is NOT an excuse!)

No response

Version of OctoPrint

1.10.1

Operating system running OctoPrint

OctoPi 0.17.0

Printer model & used firmware incl. version

Ender 3, custom firmware

Browser and version of browser, operating system running browser

Firefox 126.0

Checklist of files to include below

  • Systeminfo Bundle (always include!)
  • Contents of the JavaScript browser console (always include in cases of issues with the user interface)
  • Screenshots and/or videos showing the problem (always include in case of issues with the user interface)
  • GCODE file with which to reproduce (always include in case of issues with GCODE analysis or printing behaviour)

Additional information & file uploads

No response

@github-actions github-actions bot added the triage This issue needs triage label May 23, 2024
@foosel foosel added bug Issue describes a bug and removed triage This issue needs triage labels May 27, 2024
@foosel foosel added this to the 1.10.x milestone May 27, 2024
@foosel foosel added the done Done but not yet released label May 27, 2024
foosel added a commit that referenced this issue May 27, 2024
Needs to be done so the filters take the change into account.

See also #5021
@foosel
Copy link
Member

foosel commented May 27, 2024

Confirmed. Fix is ready for 1.10.2.

The logic for folders was broken in general as it would hide folders that have only one successful print on ANY file in them, so the logic has now changed:

  • folders will be shown as long as they have at least one file in them that has not been printed successfully yet (recursively!)
  • folders will also be shown if they contain the currently selected file, regardless of the amount of successful prints

@notoriouscarl
Copy link
Author

thanks for looking into it.

@foosel foosel closed this as completed in 81114bf Jun 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Issue describes a bug done Done but not yet released
Projects
Status: Done
Development

No branches or pull requests

2 participants