Skip to content

Fix wrong shape on propTypes.tapAction#2253

Merged
corinagum merged 4 commits intomicrosoft:masterfrom
compulim:fix-2250
Aug 1, 2019
Merged

Fix wrong shape on propTypes.tapAction#2253
corinagum merged 4 commits intomicrosoft:masterfrom
compulim:fix-2250

Conversation

@compulim
Copy link
Copy Markdown
Contributor

@compulim compulim commented Jul 31, 2019

Fixes #2250.

Also need to fix card broken command.

image

image

Changelog Entry

Description

Wrong propTypes on AdaptiveCardRenderer.js.

Specific Changes

  • Modify AdaptiveCardsRenderer.propTypes.tapAction

  • Testing Added

Copy link
Copy Markdown
Contributor

@tonyanziano tonyanziano left a comment

Choose a reason for hiding this comment

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

:shipit:

@compulim compulim changed the title Fix wrong shape on propTypes.tapAction [Do not review] Fix wrong shape on propTypes.tapAction Jul 31, 2019
@coveralls
Copy link
Copy Markdown

coveralls commented Jul 31, 2019

Coverage Status

Coverage decreased (-0.03%) to 64.958% when pulling e52a72d on compulim:fix-2250 into 95e4ecd on microsoft:master.

@compulim compulim changed the title [Do not review] Fix wrong shape on propTypes.tapAction Fix wrong shape on propTypes.tapAction Jul 31, 2019
@compulim
Copy link
Copy Markdown
Contributor Author

@corinagum This is the fix for the aria-label on TypingAnimation.

@corinagum corinagum merged commit 6169f64 into microsoft:master Aug 1, 2019
@compulim compulim deleted the fix-2250 branch August 1, 2019 17:06
compulim added a commit to compulim/BotFramework-WebChat that referenced this pull request Aug 1, 2019
* Fix wrong shape on propTypes.tapAction

* Update PR number

* Should pass language

* Pass language and rename to aria-label
compulim added a commit that referenced this pull request Aug 1, 2019
* Fix wrong shape on propTypes.tapAction

* Update PR number

* Should pass language

* Pass language and rename to aria-label
@compulim compulim mentioned this pull request Oct 25, 2019
55 tasks
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.

Wrong propTypes on AdaptiveCardRenderer.tapAction

4 participants