Skip to content

Fix Link Button#1

Open
zpriddy wants to merge 2 commits intomasterfrom
zpriddy-patch-1
Open

Fix Link Button#1
zpriddy wants to merge 2 commits intomasterfrom
zpriddy-patch-1

Conversation

@zpriddy
Copy link
Owner

@zpriddy zpriddy commented Nov 18, 2019

Button requires a string for value. To fix this use the same random_id as the action id

Summary

Fixed the minor bug in slackapi#546 where there needs to be a string value for LinkButtonElement

Requirements (place an x in each [ ])

Button requires a string for value. To fix this use the same random_id as the action id
Update test to match change to LinkButtonElement
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.

1 participant