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

Can't export training program with '/' character in title #32

Open
jm355 opened this issue Dec 3, 2020 · 2 comments
Open

Can't export training program with '/' character in title #32

jm355 opened this issue Dec 3, 2020 · 2 comments
Labels
bug Something isn't working

Comments

@jm355
Copy link

jm355 commented Dec 3, 2020

Describe the bug
A training program with a '/' character in the title can be created, however when it's exported it simply fails with no explanation as to why

To Reproduce
Steps to reproduce the behavior:

  1. Create a new training program
  2. Rename it to anything, as long as it has a '/' character. In my case it was "r/bodyweightfitness Recommended Routine"
  3. Save new training program
  4. Export training program anywhere

Expected behavior
I expect it to be able to export any training program I'm able to create. A couple ideas for how to do that would be blocking me from creating a training program with a '/' in the title or changing the way it stores/exports files so that file names may be slightly different from the program title

Additional context
Add any other context about the problem here.

Debug log
openWorkout_2020-12-03_07-09.txt

@oliexdev oliexdev added the bug Something isn't working label Dec 7, 2020
@oliexdev
Copy link
Owner

oliexdev commented Dec 7, 2020

Currently this is not possible because I create a path with the workout name and on linux the char '/' is interpreted as a subfolder.

@jm355
Copy link
Author

jm355 commented Dec 7, 2020

Yep, I figured the best solution would be to just have an error if the string contains certain characters. Or make a function that cleans the string before using it to access the file system

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants