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.

Sign up for GitHub

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

Jump to bottom

Multi-Print combine functions doesn't account for paths #1233

Open
cuesir opened this issue Sep 29, 2022 · 2 comments
Open

Multi-Print combine functions doesn't account for paths #1233

cuesir opened this issue Sep 29, 2022 · 2 comments
Labels
bug Something isn't working / good issue report

Comments

@cuesir
Copy link

cuesir commented Sep 29, 2022 •
edited

OctoFarm Version

1.6.8

OctoPrint Version(s)

1.8.3

pm2 or docker?

pm2

Operating System

Windows

Description of the problem

Multi-Print Missing File Directories even though all my octoprint instance have synchronized Gcode directories.
multi-print
file manager

Steps taken to reproduce

  1. Rebooted octofarm. -- npm run restart
  2. Looked at file manager, files and directory is there.
  3. Folder Missing only in multi-print directory search.
  4. Any new uploads to octoprint upload directory is not being reflected.
@cuesir cuesir added the bug Something isn't working / good issue report label Sep 29, 2022
NotExpectedYet added a commit that referenced this issue Oct 1, 2022
@NotExpectedYet
fix(server): normalise filenames before comparing for uniqeness
4a4465c 
Closes: #1233
NotExpectedYet added a commit that referenced this issue Oct 1, 2022
@NotExpectedYet
chore(release): 1.6.9 [skip ci]
046ea39 
### [1.6.9](v1.6.8...v1.6.9) (2022-10-01)

### ➰ What a drag! ➰

* **client:** compile client down with babel support ([46015fb](46015fb))
* **client:** improve babel config ([300ba56](300ba56))
* **main:** add git ignore for new client development path ([719b9b6](719b9b6))
* **main:** fix development version assets not been found ([5c83d17](5c83d17))
* **main:** fix eslint rules triggering on top level reserved words ([b91bce4](b91bce4))
* **main:** update dependencies ([d98cb3e](d98cb3e))

### 🔨 Bug Fix 🔨

* **client:** conflict between current operation id and main views ([1ed2977](1ed2977)), closes [#1227](#1227)
* **client:** fix current operations sorting ([4ee3517](4ee3517)), closes [#1218](#1218)
* **client:** fix dashboard resizing ([ef022a0](ef022a0))
* **client:** incorrect path for background image ([a3091e5](a3091e5)), closes [#1234](#1234)
* **client:** unable to create new folders on op client ([61d4653](61d4653)), closes [#1226](#1226)
* **server:** a failed ip match would still attempt to be read ([89e543f](89e543f))
* **server:** couldn't parse octopi token when didn't exist ([00ceee7](00ceee7))
* **server:** normalise filenames before comparing for uniqeness ([e05c984](e05c984)), closes [#1233](#1233)
* **server:** unable to detach spools from printer ([20c3489](20c3489)), closes [#1232](#1232)

### 👗 UI! 👗

* **client:** split "view" button into "info" and "gallery" ([b3a1b0f](b3a1b0f))

### 😣 Code Refactor 😣

* **client:** increase length rounding so smaller values are displayed ([548cc98](548cc98))

### ❌ Removed ❌

* **client:** disable ability to edit and save spool changes ([b584d74](b584d74)), closes [#1217](#1217)
@NotExpectedYet
Copy link
Member

🎉 This issue has been resolved in version 1.6.9 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@NotExpectedYet NotExpectedYet reopened this Oct 3, 2022
@NotExpectedYet
Copy link
Member

Still missing some structures

@NotExpectedYet NotExpectedYet changed the title Multi-Print Missing File Directories Multi-Print combine functions doesn't account for paths Oct 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working / good issue report
Projects
OctoFarm Feature Roadmap
Status: Triage
Milestone
No milestone
Development

No branches or pull requests
2 participants
@NotExpectedYet @cuesir