Fixed bugs and user inputs in site commands - Githubissues

LoginRadius / lr-cli

lr is LoginRadius on the command line. You can perform basic actions of your LoginRadius Dashboard through command line. The actions includes login, register, logout, email configuration, domain whitelisting etc.

MIT License

11 stars 12 forks source link

Fixed bugs and user inputs in site commands #33

Closed Akash-Patil closed 3 years ago

Akash-Patil commented 3 years ago

Checklist

Please note that in case of below checklist not updated accordingly, the maintainers have the right to immediately close your pull request.

[x] You created a branch from develop branch.
[x] You PR is raised on develop branch and not onmain.
[x] You have read the Contribution Guidlines before creating this PR.

Description of the Change

Added ownedAppCount to add site (auth/create-new-app) API request.
Updated user inputs for add site and add hooks commands to support users using Windows Binary/installations.

Risk Impact

Low

Verification Process

Tested these commands in Windows and MacOs systems:

lr add site
lr add hooks
lr delete site
lr delete hooks

Release Notes

Updated user inputs for add site and add hooks commands.
Modified API request to add site API.

Akash-Patil commented 3 years ago

We are using scanf here also - cmd/set/theme/theme.go, kindly check and update

In cmd/set/theme/theme.go, we are using only 1 scanf() statement. It will work fine without \n in this case.