search

nfdi4plants / ARCCommander

Tool to manage your ARCs
MIT License
11 stars 9 forks source link

[Feature Request] warning for long assay / study titles #126

Closed Brilator closed 1 year ago

Brilator commented 2 years ago

Is your feature request related to a problem? Please describe. Excel bugs with long (>31 characters) sheet names.

Describe the solution you'd like Just a quick warning to reduce titles to max. 31 chars and preferentially without specials and spaces.

Additional context

Try 

arc s add -s 'This Is a very long Title that Excel does not like'

see Excel error (at least on Mac):

"We found a problem with some content in 'isa.study.xlsx'. Do you want us to try to recover as much as we can? If you trust the source of this workbook, click Yes."

HLWeil commented 2 years ago

Hey @Brilator, good point! Besides this length constraint, also some other constraints like characters that clash with the different OS path names need to be blocked for some arguments. E.g. for windows: image

@muehlhaus, @chgarth, @omaus, maybe we could collect similar constraints here in this issue?

omaus commented 2 years ago

Hey @Brilator, good point! Besides this length constraint, also some other constraints like characters that clash with the different OS path names need to be blocked for some arguments. E.g. for windows: image

@muehlhaus, @chgarth, @omaus, maybe we could collect similar constraints here in this issue?

` (Space) is difficult on Linux and should be avoided (though it is not forbidden). Maybe replacing with_when OS is Linux? Only/andNUL` are forbidden in Linux. Don't know for Mac. @Brilator ?