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

Saveas 2 #175

Closed
wants to merge 2 commits into from
Closed

Saveas 2 #175

wants to merge 2 commits into from

Conversation

djdiskmachine
Copy link
Contributor

Rebased on master, tested and working
Save As will reboot to load console after saving.
Without calling watchdog_reboot, dialog will hang until manually rebooting.

djdiskmachine added 2 commits April 4, 2024 08:09
Mostly the work of yoyz (https://github.com/yoyz)
Save as enters a project name popup
When entering a new project name, project items will be copied to a folder with a new name.

Build error:
sources/Application/Views/ProjectView.cpp:301:5: error: duplicate case value
  301 |     case ACTION_BOOTSEL: {
      |     ^~~~
sources/Application/Views/ProjectView.cpp:278:5: note: previously used here
  278 |     case ACTION_SAVE_AS: {
      |     ^~~~
Copies all files and reboots to load menu
@maks
Copy link
Collaborator

maks commented Aug 29, 2024

@djdiskmachine Thanks again for submitting this and sorry its taken me so long to get back to this PR.
We are taking a different approach to the file system and project file handling to how LGPT previously did it so this won't be mergable with those changes in the picoTracker code base coming for version 2.

@maks maks closed this Aug 29, 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
Development

Successfully merging this pull request may close these issues.

2 participants