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

gupload: line 724: 1: Give account name #176

Closed
borland502 opened this issue Sep 30, 2021 · 1 comment · Fixed by #185
Closed

gupload: line 724: 1: Give account name #176

borland502 opened this issue Sep 30, 2021 · 1 comment · Fixed by #185
Labels
bug

Comments

@borland502
Copy link

borland502 commented Sep 30, 2021 •
edited
Loading 

12:15:28 ❯ gupload ~/.todo/todo.txt
[ Checking credentials.. ]
[  New account name:  ]
[ Info: Account names can only contain alphabets / numbers / dashes. ]

[  Given account name:   ]
/Users/foo/.google-drive-upload/bin/gupload: line 724: 1: Give account name

~  took 14s
🕙 12:26:14 ❯ gupload ~/.todo/todo.txt
[ Checking credentials.. ]
[  New account name:  ]
[ Info: Account names can only contain alphabets / numbers / dashes. ]

This is likely a usability (user error) bug, rather than a true bug given no issues submitted. However, no matter what I type for the account name Test, default, [client-id] the result is an exit before anything is setup with the error above. No, I lie, if I do client_id I do get an invalid complaint.

zsh using iterm, macos: darwin20.0

update: I got around the setup issue by using the -ca flag and things proceeded as I expected. Thanks for the script, but I had little clue why one route worked and the default route failed.
@Akianonymus
Copy link
Collaborator

I will looks into it, probably some typo.

@Akianonymus Akianonymus added the bug label Oct 2, 2021
Akianonymus added a commit to Akianonymus/google-drive-upload that referenced this issue May 15, 2022
@Akianonymus
Use grep for -ex and -in flag | Resolve labbots#176
334ecea
@Akianonymus Akianonymus linked a pull request May 16, 2022 that will close this issue
Full rewrite #185
Merged
Akianonymus added a commit to Akianonymus/google-drive-upload that referenced this issue May 16, 2022
@Akianonymus
Full rewrite
8f9eca5 
use a improved argument parser

add a -- flag, to treat all input as file or folder or drive id input

stop using set -e anywhere

add a improved config parser

fix all edgecases in authentication

commonize more code as possible

move update code to update.sh

do not edit zshrc or bashrc with the install script, instead show a help text on what to add to the zshrc or bashrc

move all files to src folder and release scripts in release folder

fix all shellcheck warnings

resolve labbots#174 labbots#176
Akianonymus added a commit to Akianonymus/google-drive-upload that referenced this issue May 16, 2022
@Akianonymus
Use grep for -ex and -in flag | Resolve labbots#176
54968c4
Akianonymus added a commit to Akianonymus/google-drive-upload that referenced this issue May 16, 2022
@Akianonymus
Full rewrite
6d3b062 
use a improved argument parser

add a -- flag, to treat all input as file or folder or drive id input

stop using set -e anywhere

add a improved config parser

fix all edgecases in authentication

commonize more code as possible

move update code to update.sh

do not edit zshrc or bashrc with the install script, instead show a help text on what to add to the zshrc or bashrc

move all files to src folder and release scripts in release folder

fix all shellcheck warnings

resolve labbots#174 labbots#176
Akianonymus added a commit to Akianonymus/google-drive-upload that referenced this issue May 16, 2022
@Akianonymus
Use grep for -ex and -in flag | Resolve labbots#176
9c914da
@Akianonymus Akianonymus mentioned this issue May 16, 2022
Merged
Akianonymus added a commit that referenced this issue Jul 14, 2022
@Akianonymus
Full rewrite
43c75ff 
use a improved argument parser

add a -- flag, to treat all input as file or folder or drive id input

stop using set -e anywhere

add a improved config parser

fix all edgecases in authentication

commonize more code as possible

move update code to update.sh

do not edit zshrc or bashrc with the install script, instead show a help text on what to add to the zshrc or bashrc

move all files to src folder and release scripts in release folder

fix all shellcheck warnings

resolve #174 #176
Akianonymus added a commit that referenced this issue Jul 14, 2022
@Akianonymus
Use grep for -ex and -in flag | Resolve #176
8bb1041
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Full rewrite Akianonymus/google-drive-upload
2 participants
@borland502 @Akianonymus