Skip to content

Added buttons#4

Open
ArmanAirapetov wants to merge 1 commit intodimonchik0036:masterfrom
ArmanAirapetov:master
Open

Added buttons#4
ArmanAirapetov wants to merge 1 commit intodimonchik0036:masterfrom
ArmanAirapetov:master

Conversation

@ArmanAirapetov
Copy link

No description provided.

Copy link
Owner

@dimonchik0036 dimonchik0036 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!
Could you add an example for a keyboard like https://github.com/dimonchik0036/vk-api/blob/master/messages_test.go#L7 and squash commits after fixing of comments?

var buttons []Button

for text, color := range textAndColorButtons {
button := new(Button)
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe it's better to create button without new? Like

button := Button{
	Action: Action{
		Type:  "text",
		Label: text,
	},
	Color: color,
}

config := NewMessage(NewDstFromUserID(id), message)
values := addParameters(config)

keyboard := new(Keyboard)
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also remove new.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants