Closed dimerman closed 1 year ago
Will need to add:
Slack::BlockKit::Elements::UrlTextInput
(or should it be URLTextInput
?)Slack::BlockKit::Layout::Input#url_text_input
It seems that at the same time email_text_input
, number_input
, and datetimepicker
were added to the BlockKit, which we should probably also support.
@CGA1123 how can I help with this?
Hey @dimerman! If you want to pick this up, then feel free to start hacking away on adding these blocks.
You should hopefully be able to get enough context from the currently implemented blocks to get started, but let me know if there are any issues.
Only thing that would be nice would be to split up the addition of these blocks into separate PRs to make it easier to follow what's being added 😄
@CGA1123 maybe create dedicated issues here on github, so there's no duplicate work? I cannot start working on this immediately, but I also don't want to become a blocker.
if/when somebody starts work on this, they can assign the issue to themselves.
Yeah that makes sense. I'll split up the individual blocks into separate issues.
Done: #161 #162
@CGA1123 PR #164 ready for your review
Docs: https://api.slack.com/reference/block-kit/block-elements#url
Implement:
Slack::BlockKit::Elements::UrlTextInput
Slack::BlockKit::Layout::Input#url_text_input