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

feat(move): implement #1209 #1524

Closed

Conversation

intothevoid1986
Copy link

Description

This PR adds the feature reqested in FR #1209.
Code is recycled from "rename" function, properly relabelled to improve UX.

The feature impact ConfigFilePanel, GCodeFilePanel and Status/GcodefilesPanel

Related Tickets & Documents

Fixes #1209

Mobile & Desktop Screenshots/Recordings

Desktop view GCode Files

Schermata del 2023-08-19 14-19-51

Desktop view Config Files

Schermata del 2023-08-19 14-21-32

Mobile view GCode Files

Schermata del 2023-08-19 14-20-34

Mobile view Config Files

Schermata del 2023-08-19 14-21-02

Move-to Dialog

Schermata del 2023-08-19 14-22-02

[optional] Are there any post-deployment tasks we need to perform?

@github-actions
Copy link
Contributor

Language file analysis report:

File Missing Keys Unused Keys
en.json 0 4

@github-actions
Copy link
Contributor

Language file analysis report:

File Missing Keys Unused Keys
en.json 0 4
it.json 380 9

@github-actions
Copy link
Contributor

Language file analysis report:

File Missing Keys Unused Keys
en.json 0 4
it.json 375 9

@github-actions
Copy link
Contributor

Language file analysis report:

File Missing Keys Unused Keys
en.json 0 4
it.json 375 9

@meteyou
Copy link
Member

meteyou commented Aug 27, 2023

Sorry for the late response.

I think you misunderstood the FR. In this PR, you added the "rename" function with new names again. But the user wanted to select several files and move these simultaneously to different directory. In the background, the Moonraker "move" API is used, but it should not change the name but the location.

@meteyou
Copy link
Member

meteyou commented Oct 1, 2023

i will close this PR, because of no response...

@meteyou meteyou closed this Oct 1, 2023
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.

Gcode file page - Move to & Move to with multiple selected
3 participants