Add Login Manage Functions #182

Merged
6543 merged 11 commits from 6543/tea:login-manage into master 2020-09-23 14:23:32 +00:00
Owner

close #160

close #160
6543 added 2 commits 2020-09-16 05:22:24 +00:00
manage Active login via CI
All checks were successful
continuous-integration/drone/pr Build is passing
1b26377558
6543 added the
kind
feature
status/needs-reviews
labels 2020-09-16 05:23:30 +00:00
6543 added this to the v0.5.0 milestone 2020-09-16 05:23:34 +00:00
6543 added 1 commit 2020-09-16 13:49:09 +00:00
Merge branch 'master' into login-manage
All checks were successful
continuous-integration/drone/pr Build is passing
df1909261f
lunny reviewed 2020-09-17 05:34:59 +00:00
Dismissed
@ -33,2 +36,4 @@
}
// cmdLoginEdit represents to login a gitea server.
var cmdLoginEdit = cli.Command{
Owner

It seems these are unrelated changes?

It seems these are unrelated changes?
Author
Owner

where ?!?

where ?!?
lunny marked this conversation as resolved
6543 added 1 commit 2020-09-17 12:50:34 +00:00
Merge branch 'master' into login-manage
All checks were successful
continuous-integration/drone/pr Build is passing
d3538d5d8c
6543 added 1 commit 2020-09-17 19:37:58 +00:00
Merge branch 'master' into login-manage
All checks were successful
continuous-integration/drone/pr Build is passing
aca5995cd7
lunny approved these changes 2020-09-18 00:19:40 +00:00
Dismissed
6543 added 1 commit 2020-09-19 18:54:01 +00:00
Merge branch 'master' into login-manage
All checks were successful
continuous-integration/drone/pr Build is passing
a4c57d95f6
Owner

Maybe default is a better name for that.

Maybe `default` is a better name for that.
Author
Owner

since the Active field was not configurable with tea before I can change the name ... just have to informe users who manualy have set it ...

-> kind/breaking

since the **Active** field was not configurable with tea before I can change the name ... just have to informe users who manualy have set it ... -> kind/breaking
6543 added the
kind/breaking
label 2020-09-21 00:51:28 +00:00
6543 added 1 commit 2020-09-21 00:57:52 +00:00
rename Active to Default
All checks were successful
continuous-integration/drone/pr Build is passing
49201ce995
Member

@6543 I'd prefer default over active

@6543 I'd prefer `default` over `active`
6543 added 1 commit 2020-09-21 06:19:01 +00:00
Merge branch 'master' into login-manage
All checks were successful
continuous-integration/drone/pr Build is passing
d9fde281aa
Author
Owner

@noerw already changed :)

@noerw already changed :)
6543 added 1 commit 2020-09-23 10:42:41 +00:00
Merge branch 'master' into login-manage
All checks were successful
continuous-integration/drone/pr Build is passing
0fcb17ed25
noerw requested changes 2020-09-23 10:59:09 +00:00
Dismissed
cmd/login.go Outdated
@ -35,0 +45,4 @@
}
func runLoginEdit(ctx *cli.Context) error {
open.Run(yamlConfigPath)
Member
please use `open.Start()` (https://github.com/skratchdot/open-golang#func-startinput-string-error)
6543 marked this conversation as resolved
cmd/login.go Outdated
@ -35,0 +53,4 @@
var cmdLoginSetDefault = cli.Command{
Name: "default",
Usage: "Set Default Login",
Description: `Set Default Login`,
Member

"Get or set default Login"?

"Get or set default Login"?
@ -35,0 +68,4 @@
if err != nil {
return err
}
fmt.Printf("Activ Login: %s\n", l.Name)
Member

Activ -> Active

Activ -> Active
6543 marked this conversation as resolved
6543 added 2 commits 2020-09-23 14:17:23 +00:00
suggestions ...
All checks were successful
continuous-integration/drone/pr Build is passing
5cff99c8a0
Author
Owner

@noerw done

@noerw done
noerw approved these changes 2020-09-23 14:21:36 +00:00
Dismissed
6543 merged commit eacf1be066 into master 2020-09-23 14:23:30 +00:00
6543 removed the
status/needs-reviews
label 2020-09-23 14:23:38 +00:00
6543 deleted branch login-manage 2020-09-23 14:23:46 +00:00
Sign in to join this conversation.
No description provided.