Added missing toggle to Accordion

[ml054]

Added missing toggle property to accordion. W/o that we have prop types issue:

 Failed prop type: The prop `toggle` is marked as required in `Accordion`, but its value is `undefined`.

See here:

reactstrap/src/Accordion.js

Line 25 in ee18f48

toggle: PropTypes.func.isRequired,

• Bug fix
• New feature
• Chore
• Breaking change
• There is an open issue which this change addresses
• I have read the CONTRIBUTING document.
• My commits follow the Git Commit Guidelines
• My code follows the code style of this project.
• My change requires a change to the documentation.
  • I have updated the documentation accordingly.
• My change requires a change to Typescript typings.
  • I have updated the typings accordingly.
• I have added tests to cover my changes.
• All new and existing tests passed.

[davidacevedo]

Thanks for the contribution!

[Samettkaya]

Hello, the problem here is version 9.1.5, which version does it work in?

[ml054]

PR is still open... :) So if might work if you manually build version with this PR

[ml054]

Any update on that? I wish I could have clean console during test runs. Can you help me with that?

[ml054]

?

[ml054]

Woo hoo, looks like it was already merged. When can I expect this in stable release? We are so close to clean console output. Let's make that happen. :)

[kpull1]

It's been a year since this problem has been fixed and it's not released yet. Is anybody using the Accordion?
I've tried by myself and this patch breaks the UncontrolledAccordion.
It lacks to add the toggle in the Omit<AccordionProps, 'open' | 'toggle'>